[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/carbondata/pull/1984 ---

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169860093 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,717 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169859849 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,689 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169858244 --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/page/SafeDecimalColumnPage.java --- @@ -217,6 +217,6 @@ public void

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169641109 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169638631 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,717 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169635879 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169635618 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169632391 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169629728 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169629127 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169628918 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169628603 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/SegmentIndexFileStore.java --- @@ -161,9 +212,13 @@ private void

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169628343 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/SegmentIndexFileStore.java --- @@ -79,6 +87,44 @@ public void

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169628065 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletDataMap.java --- @@ -660,22 +650,23 @@ public boolean

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169627883 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/TableBlockIndexUniqueIdentifier.java --- @@ -17,91 +17,59 @@ package

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169627561 --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/page/SafeDecimalColumnPage.java --- @@ -217,6 +217,6 @@ public void

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169627320 --- Diff: core/src/main/java/org/apache/carbondata/core/datamap/Segment.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169627375 --- Diff: core/src/main/java/org/apache/carbondata/core/datamap/Segment.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169627117 --- Diff: core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java --- @@ -2449,5 +2496,40 @@ public static String encodeToString(byte[]

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169626625 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169626278 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169620760 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletDataMapFactory.java --- @@ -185,26 +212,27 @@ public void

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169619961 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/PartitionSpec.java --- @@ -0,0 +1,80 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-21 Thread ravipesala
Github user ravipesala commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169619680 --- Diff: core/src/main/java/org/apache/carbondata/core/datamap/Segment.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169544845 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169544627 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169544464 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169543473 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169543287 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169542459 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169518712 --- Diff: core/src/main/java/org/apache/carbondata/core/datamap/Segment.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169540773 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/SegmentIndexFileStore.java --- @@ -79,6 +87,44 @@ public void

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169537887 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletDataMap.java --- @@ -660,22 +650,23 @@ public boolean

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169518845 --- Diff: core/src/main/java/org/apache/carbondata/core/datamap/Segment.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to the Apache Software

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169540997 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/SegmentIndexFileStore.java --- @@ -161,9 +212,13 @@ private

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169520945 --- Diff: core/src/main/java/org/apache/carbondata/core/datastore/page/SafeDecimalColumnPage.java --- @@ -217,6 +217,6 @@ public void

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread manishgupta88
Github user manishgupta88 commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169522942 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/TableBlockIndexUniqueIdentifier.java --- @@ -17,91 +17,59 @@

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169413537 --- Diff: core/src/main/java/org/apache/carbondata/core/util/CarbonUtil.java --- @@ -2449,5 +2496,40 @@ public static String encodeToString(byte[]

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169411226 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169402456 --- Diff: core/src/main/java/org/apache/carbondata/core/metadata/SegmentFileStore.java --- @@ -0,0 +1,690 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169399178 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletDataMapFactory.java --- @@ -185,26 +212,27 @@ public void

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169392444 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/PartitionSpec.java --- @@ -0,0 +1,80 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169392124 --- Diff: core/src/main/java/org/apache/carbondata/core/indexstore/PartitionSpec.java --- @@ -0,0 +1,80 @@ +/* + * Licensed to the Apache

[GitHub] carbondata pull request #1984: [CARBONDATA-2187] Partition restructure

2018-02-20 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/carbondata/pull/1984#discussion_r169380463 --- Diff: core/src/main/java/org/apache/carbondata/core/datamap/Segment.java --- @@ -0,0 +1,93 @@ +/* + * Licensed to the Apache Software