carbondata-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From lionelcao <>
Subject [GitHub] carbondata pull request #1125: [CarbonData-1250] change default partition id...
Date Mon, 03 Jul 2017 10:02:01 GMT
GitHub user lionelcao opened a pull request:

    [CarbonData-1250] change default partition id & Add TaskId group in partitionInfo

    This PR change the default partition id from Max to 0 and add taskId management related
code to support later add(split)/drop(merge) partition operations.
    1. change default partition id from Max to 0
    2. add taskIdInPartitionOrder in partitionInfo to manage the taskId to partition map
    3. remove some useless file
    4. extract some common used function to PartitionUtils
    5. change CarbonInputFormat to use correct taskId
    6. modify test case according to the taskId change.
    7. support range info in ascending order validation
    8. fix some typo

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

    $ git pull carbon_910_13

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

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

    This closes #1125
commit 83d134919b8962ac3d6269f7c910666527c3a4c5
Author: lionelcao <>
Date:   2017-06-29T16:00:32Z

    [CARBONDATA-1250] change default partition id from Max to 0

commit 5fb7447cc03b8204e98e90129037eee828912bf8
Author: lionelcao <>
Date:   2017-06-29T16:06:30Z

    extract common function

commit fd928d8476b0907dd1cb0347803775b495fe5f74
Author: lionelcao <>
Date:   2017-06-30T15:05:06Z

    add taskid group in partition info and change query filter code
    remove partition statistics for now
    fix style error
    empty commit
    add taskid group in partitionInfo
    remove unrelated code
    fix style error
    fix style error
    fix bug: getSplits error for non-partition table
    change import order
    remove unrelated code and test CI
    change import order
    change query filter code according to default partition id changing
    thrift converter
    remove useless comment
    remove unrelated code


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 or file a JIRA ticket
with INFRA.

View raw message