[GitHub] carbondata pull request #1325: [CARBONDATA-1437] Fixed bug for incorrect exc...

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

[GitHub] carbondata pull request #1325: [CARBONDATA-1437] Fixed bug for incorrect exc...

qiuchenjian-2
GitHub user ksimar opened a pull request:

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

    [CARBONDATA-1437] Fixed bug for incorrect exception message when loading to table having property 'bucketnumber' set to 0.

    1. In this PR, Table creation would not be allowed, if the table property 'number of buckets' is set to 0 in create command.
    2. Added unit test case for it as well in class TableBucketingTestCase.scala


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

    $ git pull https://github.com/ksimar/incubator-carbondata bugfix/CARBONDATA-1437

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

    https://github.com/apache/carbondata/pull/1325.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 #1325
   
----
commit 6bd1717070612cd330aaca087e7af6000152d588
Author: ksimar <[hidden email]>
Date:   2017-09-05T12:13:17Z

    Fixed Bug for incorrect exception message in loading to table created with property 'number of buckets' to 0.
    Now, Table will not be created if the property 'number of buckets' is set to 0 in create command.

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #1325: [CARBONDATA-1437] Fixed bug for incorrect exception ...

qiuchenjian-2
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1325
 
    SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/533/



---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #1325: [CARBONDATA-1437] Fixed bug for incorrect exception ...

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

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


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #1325: [CARBONDATA-1437] Fixed bug for incorrect exc...

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

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


---