Github user asfgit closed the pull request at:
https://github.com/apache/carbondata/pull/2823
---
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r228106032
--- Diff:
integration/spark-datasource/src/main/spark2.1andspark2.2/org/apache/spark/sql/CarbonVectorProxy.java
---
@@ -150,127 +140,189 @@ public
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r228100832
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/scanner/LazyBlockletLoad.java
---
@@ -0,0 +1,158 @@
+/*
+ * Licensed to the
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r228099605
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/result/vector/impl/directread/AbstractCarbonColumnarVector.java
---
@@ -130,4 +131,9
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r228099219
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/result/BlockletScannedResult.java
---
@@ -145,6 +147,8 @@
protected
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r228096779
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -48,7 +48,11
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r227621527
--- Diff:
integration/spark-datasource/src/main/spark2.1andspark2.2/org/apache/spark/sql/CarbonVectorProxy.java
---
@@ -150,127 +140,189 @@ public
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r227620816
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/scanner/LazyBlockletLoad.java
---
@@ -0,0 +1,158 @@
+/*
+ * Licensed to the
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r227620665
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/result/vector/impl/directread/AbstractCarbonColumnarVector.java
---
@@ -130,4 +131,9 @@
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r227620364
--- Diff:
core/src/main/java/org/apache/carbondata/core/scan/result/BlockletScannedResult.java
---
@@ -145,6 +147,8 @@
protected
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r227620200
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -48,7 +48,11 @@
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226867984
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/adaptive/AdaptiveDeltaIntegralCodec.java
---
@@ -272,5 +295,169 @@
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226867910
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/adaptive/AdaptiveDeltaFloatingCodec.java
---
@@ -225,6 +238,71 @@
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226867851
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226867498
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226867364
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226867324
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226867290
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226867258
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/SafeVariableLengthDimensionDataChunkStore.java
---
@@ -91,6
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226549503
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226549110
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226547597
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/SafeVariableLengthDimensionDataChunkStore.java
---
@@ -91,6
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226548215
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226548594
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226550873
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/adaptive/AdaptiveDeltaIntegralCodec.java
---
@@ -272,5 +295,169 @@
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226547892
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -0,0 +1,274
Github user xuchuanyin commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2823#discussion_r226550593
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/adaptive/AdaptiveDeltaFloatingCodec.java
---
@@ -225,6 +238,71 @@
GitHub user ravipesala opened a pull request:
https://github.com/apache/carbondata/pull/2823
[CARBONDATA-3015] Support Lazy load in carbon vector
This PR depends on PR https://github.com/apache/carbondata/pull/2822
Even though we prune the pages as per min/max there is a
28 matches
Mail list logo