Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87716151
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java ---
@@ -130,41 +130,27 @@ public static CarbonTable
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87716124
--- Diff:
integration/spark/src/main/scala/org/apache/carbondata/spark/rdd/CarbonScanRDD.scala
---
@@ -244,68 +195,40 @@ class
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87716097
--- Diff:
processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java
---
@@ -97,15 +91,14 @@ public long
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87716071
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java ---
@@ -224,42 +221,29 @@ private void
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87716028
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java ---
@@ -224,42 +221,29 @@ private void
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87714121
--- Diff:
integration/spark/src/main/scala/org/apache/spark/mapreduce/CarbonHadoopMapReduceUtil.scala
---
@@ -0,0 +1,25 @@
+/*
+ *
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87713382
--- Diff:
integration/spark/src/main/scala/org/apache/spark/mapreduce/CarbonHadoopMapReduceUtil.scala
---
@@ -0,0 +1,25 @@
+/*
+ *
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87712886
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java ---
@@ -224,42 +221,29 @@ private void
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87712366
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java ---
@@ -130,41 +130,27 @@ public static CarbonTable
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87540836
--- Diff:
processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java
---
@@ -177,6 +178,13 @@ public
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r87540816
--- Diff:
processing/src/main/java/org/apache/carbondata/lcm/status/SegmentStatusManager.java
---
@@ -177,6 +178,13 @@ public
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r86786637
--- Diff:
core/src/main/java/org/apache/carbondata/core/carbon/datastore/block/Distributable.java
---
@@ -16,10 +16,12 @@
*/
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r86786673
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java ---
@@ -311,80 +278,6 @@ private void
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r86393676
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormat.java ---
@@ -311,80 +278,6 @@ private void
Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r86391469
--- Diff:
core/src/main/java/org/apache/carbondata/core/carbon/datastore/block/Distributable.java
---
@@ -16,10 +16,12 @@
*/
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r86214621
--- Diff:
integration/spark/src/main/java/org/apache/carbondata/spark/load/CarbonLoaderUtil.java
---
@@ -953,66 +959,6 @@ public static
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r86211496
--- Diff:
core/src/main/java/org/apache/carbondata/scan/filter/FilterExpressionProcessor.java
---
@@ -352,4 +354,18 @@ private
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r86058166
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputSplit.java ---
@@ -22,28 +22,44 @@
import java.io.DataOutput;
Github user QiangCai commented on a diff in the pull request:
https://github.com/apache/incubator-carbondata/pull/262#discussion_r86058188
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputSplit.java ---
@@ -22,28 +22,44 @@
import java.io.DataOutput;
19 matches
Mail list logo