Mailing-List: contact dev-help@phoenix.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@phoenix.apache.org
Date: Wed, 25 Oct 2017 21:22:01 +0000 (UTC)
From: "ASF GitHub Bot (JIRA)" <jira@apache.org>
To: dev@phoenix.apache.org
Message-ID: <JIRA.13060807.1490994869000.61608.1508966521183@Atlassian.JIRA>
In-Reply-To: <JIRA.13060807.1490994869000@Atlassian.JIRA>
References: <JIRA.13060807.1490994869000@Atlassian.JIRA> <JIRA.13060807.1490994869966@jira-lw-us.apache.org>
Subject: [jira] [Commented] (PHOENIX-3757) System mutex table not being
 created in SYSTEM namespace when namespace mapping is enabled
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Wed, 25 Oct 2017 21:22:10 -0000


    [ https://issues.apache.org/jira/browse/PHOENIX-3757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16219540#comment-16219540 ] 

ASF GitHub Bot commented on PHOENIX-3757:
-----------------------------------------

Github user aertoria commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/277#discussion_r146990595
  
    --- Diff: phoenix-core/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java ---
    @@ -2526,8 +2541,14 @@ private void createOtherSystemTables(PhoenixConnection metaConnection) throws SQ
             try {
                 metaConnection.createStatement().execute(QueryConstants.CREATE_FUNCTION_METADATA);
             } catch (TableAlreadyExistsException ignore) {}
    +        // We catch TableExistsException in createSysMutexTable() and ignore it. Hence we will also ignore IOException here.
    +        // SYSTEM.MUTEX table should not be exposed to user. Hence it is directly created and used via HBase API.
    +        // Using 'CREATE TABLE' statement will add entries to SYSTEM.CATALOG table, which should not happen.
    +        try {
    +            createSysMutexTable(hBaseAdmin, ConnectionQueryServicesImpl.this.getProps());
    +        } catch (IOException ignore) {}
    --- End diff --
    
    I think in those case you can just ignore the exception that you want to tolerate. So at least SQLException should be bubbled up.
    

> System mutex table not being created in SYSTEM namespace when namespace mapping is enabled
> ------------------------------------------------------------------------------------------
>
>                 Key: PHOENIX-3757
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-3757
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: Josh Elser
>            Assignee: Karan Mehta
>            Priority: Critical
>              Labels: namespaces
>             Fix For: 4.13.0
>
>         Attachments: PHOENIX-3757.001.patch, PHOENIX-3757.002.patch, PHOENIX-3757.003.patch
>
>
> Noticed this issue while writing a test for PHOENIX-3756:
> The SYSTEM.MUTEX table is always created in the default namespace, even when {{phoenix.schema.isNamespaceMappingEnabled=true}}. At a glance, it looks like the logic for the other system tables isn't applied to the mutex table.


--
This message was sent by Atlassian JIRA
(v6.4.14#64029)