GitHub user kunal642 opened a pull request:
https://github.com/apache/incubator-carbondata/pull/825 [CARBONDATA-961] Added condition to skip sort step for Rename Table and DataTypeChange
Earlier for all the alter operations the flow was going to sort step during compaction. But during Rename Table and DataTypeChange sorting is not required.
Solution:
Added a map which contains the timestamp of the operation and what operation was performed. Based on this during compaction we judge whether to perform sorting or not.
You can merge this pull request into a Git repository by running:
$ git pull
https://github.com/kunal642/incubator-carbondata skip_sort_step
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-carbondata/pull/825.patchTo close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #825
----
commit acf2210d653e5cc1e35a568d1156f7ac94b3eece
Author: kunal642 <
[hidden email]>
Date: 2017-04-20T11:10:44Z
added condition to skip sort step
commit 6bacff42f79c6ccaf6f3147cbc29685a263c7a57
Author: kunal642 <
[hidden email]>
Date: 2017-04-20T13:58:48Z
removed unnecessary .show call in test cases
----
---
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.
---
Locked
