[GitHub] incubator-carbondata pull request #116: [CARBONDATA-198] System level lock f...

2016-09-01 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/116#discussion_r77127755 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -297,15 +366,144 @@ object

[GitHub] incubator-carbondata pull request #116: [CARBONDATA-198] System level lock f...

2016-09-01 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/116#discussion_r77128213 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -297,15 +366,144 @@ object

[GitHub] incubator-carbondata pull request #116: [CARBONDATA-198] System level lock f...

2016-09-01 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/116#discussion_r77129093 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -333,7 +531,7 @@ object

[GitHub] incubator-carbondata pull request #137: [CARBONDATA-222] Handled query issue...

2016-09-08 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/137#discussion_r77950155 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/datastore/impl/btree/BTreeDataRefNodeFinder.java --- @@ -240,9 +240,16

[GitHub] incubator-carbondata pull request #140: [CARBONDATA-227] In block distributi...

2016-09-08 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/140#discussion_r77950753 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonScanRDD.scala --- @@ -73,7 +73,7 @@ class CarbonScanRDD[V

[GitHub] incubator-carbondata pull request #139: [CARBONDATA-224]Fixed data mismatch ...

2016-09-07 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/139#discussion_r77947424 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/CarbonSqlParser.scala --- @@ -864,8 +864,8 @@ class CarbonSqlParser

[GitHub] incubator-carbondata pull request #111: [CARBONDATA-194] ArrayIndexOfBoundEx...

2016-08-31 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/111#discussion_r76936740 --- Diff: processing/src/main/java/org/apache/carbondata/processing/csvreaderstep/UnivocityCsvParser.java --- @@ -104,6 +108,20 @@ public

[GitHub] incubator-carbondata pull request #158: [CARBONDATA-241]Fixed out of memory ...

2016-09-15 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/158#discussion_r79003748 --- Diff: processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java --- @@ -102,6 +91,60 @@ public long

[GitHub] incubator-carbondata pull request #158: [CARBONDATA-241]Fixed out of memory ...

2016-09-17 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/158#discussion_r79290196 --- Diff: hadoop/src/test/java/org/apache/carbondata/hadoop/ft/CarbonInputMapperTest.java --- @@ -129,6 +132,37 @@ private int

[GitHub] incubator-carbondata pull request #158: [CARBONDATA-241]Fixed out of memory ...

2016-09-17 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/158#discussion_r79290267 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java --- @@ -706,8 +725,9 @@ private String getUpdateExtension

[GitHub] incubator-carbondata pull request #158: [CARBONDATA-241]Fixed out of memory ...

2016-09-17 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/158#discussion_r79290374 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonScanRDD.scala --- @@ -102,7 +115,7 @@ class CarbonScanRDD[V

[GitHub] incubator-carbondata pull request #194: [CARBONDATA-270] Double data type va...

2016-09-23 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/194#discussion_r80221140 --- Diff: core/src/main/java/org/apache/carbondata/scan/filter/FilterUtil.java --- @@ -1401,8 +1401,7 @@ public static void logError(Throwable

[GitHub] incubator-carbondata pull request #190: [CARBONDATA-268]Improve carbonoptimi...

2016-09-23 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/190#discussion_r80220071 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/optimizer/CarbonOptimizer.scala --- @@ -68,9 +70,50 @@ object CarbonOptimizer

[GitHub] incubator-carbondata pull request #190: [CARBONDATA-268]Improve carbonoptimi...

2016-09-23 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/190#discussion_r80219576 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/optimizer/CarbonOptimizer.scala --- @@ -87,20 +130,32 @@ class

[GitHub] incubator-carbondata pull request #190: [CARBONDATA-268]Improve carbonoptimi...

2016-09-23 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/190#discussion_r80219989 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/optimizer/CarbonOptimizer.scala --- @@ -68,9 +70,50 @@ object CarbonOptimizer

[GitHub] incubator-carbondata pull request #190: [CARBONDATA-268]Improve carbonoptimi...

2016-09-23 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/190#discussion_r80220168 --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/optimizer/CarbonOptimizer.scala --- @@ -397,20 +452,38 @@ class

[GitHub] incubator-carbondata pull request #155: [CARBONDATA-239] Handling the except...

2016-09-16 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/155#discussion_r79208835 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -590,16 +605,25 @@ object

[GitHub] incubator-carbondata pull request #158: [CARBONDATA-241]Fixed out of memory ...

2016-09-17 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/158#discussion_r79288466 --- Diff: core/src/main/java/org/apache/carbondata/core/carbon/datastore/BlockIndexStore.java --- @@ -260,11 +295,29 @@ public void

[GitHub] incubator-carbondata pull request #161: [CARBONDATA-246] compaction is wrong...

2016-09-17 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/161#discussion_r79289428 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonMergerRDD.scala --- @@ -259,6 +253,9 @@ class

[GitHub] incubator-carbondata pull request #161: [CARBONDATA-246] compaction is wrong...

2016-09-17 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/161#discussion_r79289289 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonMergerRDD.scala --- @@ -102,6 +102,11 @@ class

[GitHub] incubator-carbondata pull request #168: [CARBONDATA-251] making the auto com...

2016-09-17 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/168#discussion_r79287073 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonDataRDDFactory.scala --- @@ -623,14 +623,7 @@ object

[GitHub] incubator-carbondata pull request #168: [CARBONDATA-251] making the auto com...

2016-09-17 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/168#discussion_r79287056 --- Diff: core/src/main/java/org/apache/carbondata/core/constants/CarbonCommonConstants.java --- @@ -920,7 +920,7 @@ * Default value

[GitHub] incubator-carbondata pull request #158: [CARBONDATA-241]Fixed out of memory ...

2016-09-16 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/158#discussion_r79222167 --- Diff: hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java --- @@ -101,6 +106,8 @@ //comma separated list

[GitHub] incubator-carbondata pull request #189: [CARBONDATA-267] Set block_size for ...

2016-10-02 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/189#discussion_r81474992 --- Diff: format/src/main/thrift/schema.thrift --- @@ -124,6 +124,7 @@ struct TableSchema{ 1: required string table_id; // ID used

[GitHub] incubator-carbondata pull request #207: [CARBONDATA-283] VT enhancement for ...

2016-11-03 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/207#discussion_r86347851 --- Diff: processing/src/main/java/org/apache/carbondata/processing/util/CarbonTableStatusUtil.java --- @@ -0,0 +1,92

[GitHub] incubator-carbondata pull request #237: [CARBONDATA-317] - CSV having only s...

2016-10-17 Thread gvramana
Github user gvramana commented on a diff in the pull request: https://github.com/apache/incubator-carbondata/pull/237#discussion_r83584249 --- Diff: integration/spark/src/main/scala/org/apache/carbondata/spark/csv/CarbonCsvRelation.scala --- @@ -148,6 +150,10 @@ case class