[GitHub] carbondata pull request #973: [WIP] Multi User load on same table is failing...

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #973: [WIP] Multi User load on same table is failing...

qiuchenjian-2
GitHub user nareshpr opened a pull request:

    https://github.com/apache/carbondata/pull/973

    [WIP] Multi User load on same table is failing with NullPointerException.

    User1 in beeline terminal1 performs below operation
    Create table t1
    load data into t1
    User2(Does not have privilege to read t1) in beeline terminal2 performs below operation
    load data into t1 -- This command will fail for table folder permission and carbonTable will be removed from CarbonMetadata
    User1 in beeline terminal1 performs below operation
    load data into t1 -- This command will throw NullPointerException for carbonTable.
    from CarbonMetadata.
    This is corrected by fetching the CarbonTable from CarbonMetastore of corresponding Session.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/nareshpr/incubator-carbondata ConcurrentOperationOnSameTable

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/carbondata/pull/973.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #973
   
----
commit 7c3020d093ea8e79a55e110b4b2b89f6e5f5fc3a
Author: nareshpr <[hidden email]>
Date:   2017-05-30T09:18:10Z

    MultiClient Load is failing

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #973: [WIP] Multi User load on same table is failing with N...

qiuchenjian-2
Github user nareshpr commented on the issue:

    https://github.com/apache/carbondata/pull/973
 
    retest this please


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #973: [WIP] Multi User load on same table is failing with N...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user nareshpr commented on the issue:

    https://github.com/apache/carbondata/pull/973
 
    retest this please


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #973: [CARBONDATA-1107] Multi User load on same table is fa...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user gvramana commented on the issue:

    https://github.com/apache/carbondata/pull/973
 
    LGTM


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #973: [CARBONDATA-1107] Multi User load on same table is fa...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user gvramana commented on the issue:

    https://github.com/apache/carbondata/pull/973
 
    retest this please


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #973: [CARBONDATA-1107] Multi User load on same table is fa...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/973
 
    Build Success with Spark 1.6.2, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder/2119/



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #973: [CARBONDATA-1107] Multi User load on same tabl...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user asfgit closed the pull request at:

    https://github.com/apache/carbondata/pull/973


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [hidden email] or file a JIRA ticket
with INFRA.
---