Mailing-List: contact issues-help@carbondata.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@carbondata.apache.org
From: lionelcao <git@git.apache.org>
To: issues@carbondata.apache.org
Reply-To: issues@carbondata.apache.org
Message-ID: <git-pr-1125-carbondata@git.apache.org>
Subject: [GitHub] carbondata pull request #1125: [CarbonData-1250] change default partition id...
Content-Type: text/plain
Date: Mon,  3 Jul 2017 10:02:01 +0000 (UTC)
archived-at: Mon, 03 Jul 2017 10:02:03 -0000

GitHub user lionelcao opened a pull request:

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

    [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 https://github.com/lionelcao/carbondata carbon_910_13

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

    https://github.com/apache/carbondata/pull/1125.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 #1125
    
----
commit 83d134919b8962ac3d6269f7c910666527c3a4c5
Author: lionelcao <whucaolu@gmail.com>
Date:   2017-06-29T16:00:32Z

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

commit 5fb7447cc03b8204e98e90129037eee828912bf8
Author: lionelcao <whucaolu@gmail.com>
Date:   2017-06-29T16:06:30Z

    extract common function

commit fd928d8476b0907dd1cb0347803775b495fe5f74
Author: lionelcao <whucaolu@gmail.com>
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---