GitHub user ravipesala opened a pull request:
https://github.com/apache/carbondata/pull/1099
[CARBONDATA-1232] Datamap implementation for Blocklet
Bypassed btree implementations and used datamap to get the pruned
blocklets. Only driver side datamap is maintained and all required inf
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125539784
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/AbstractTableDataMap.java
---
@@ -26,28 +27,29 @@
* DataMap at the table level
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125539854
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -30,7 +31,7 @@
--- End diff --
pl
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125540172
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -69,20 +70,20 @@ public TableDataMap getDataMap(Abso
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125539929
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -92,9 +93,9 @@ public TableDataMap
createTableDataM
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125539888
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -30,7 +31,7 @@
private static DataMapSt
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125540590
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletTableMap.java
---
@@ -0,0 +1,123 @@
+/*
+ * Licensed t
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125541084
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapType.java ---
@@ -20,7 +20,7 @@
* Datamap type
*/
public enum
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125541188
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -30,7 +31,7 @@
private static DataMapSt
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125542074
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/AbstractTableDataMap.java
---
@@ -84,11 +121,11 @@
* @param filterExp
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125543490
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/row/DataMapRow.java ---
@@ -0,0 +1,88 @@
+/*
+ * Licensed to the Apache Softw
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125546037
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletTableMap.java
---
@@ -0,0 +1,123 @@
+/*
+ * Licensed t
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125546095
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletTableMap.java
---
@@ -0,0 +1,123 @@
+/*
+ * Licensed t
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125546189
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletTableMap.java
---
@@ -0,0 +1,123 @@
+/*
+ * Licensed t
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125546253
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormatNew.java ---
@@ -0,0 +1,566 @@
+/*
+ * Licensed to the Apache Softw
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r125546316
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormatNew.java ---
@@ -0,0 +1,566 @@
+/*
+ * Licensed to the Apache Softw
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126063998
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/AbstractTableDataMap.java
---
@@ -26,28 +27,29 @@
* DataMap at the table le
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126064056
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -30,7 +31,7 @@
--- End diff --
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126064109
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -30,7 +31,7 @@
private static DataMa
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126090143
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -69,20 +70,20 @@ public TableDataMap getDataMap(A
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126093030
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -92,9 +93,9 @@ public TableDataMap
createTableDa
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126093275
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletTableMap.java
---
@@ -0,0 +1,123 @@
+/*
+ * License
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126093338
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapType.java ---
@@ -20,7 +20,7 @@
* Datamap type
*/
public e
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126093417
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletTableMap.java
---
@@ -0,0 +1,123 @@
+/*
+ * License
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126093367
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/AbstractTableDataMap.java
---
@@ -84,11 +121,11 @@
* @param filterExp
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126093455
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletTableMap.java
---
@@ -0,0 +1,123 @@
+/*
+ * License
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126093505
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormatNew.java ---
@@ -0,0 +1,566 @@
+/*
+ * Licensed to the Apache So
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126093559
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/CarbonInputFormatNew.java ---
@@ -0,0 +1,566 @@
+/*
+ * Licensed to the Apache So
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126093470
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/blockletindex/BlockletTableMap.java
---
@@ -0,0 +1,123 @@
+/*
+ * License
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126420383
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -69,32 +72,54 @@ public TableDataMap getDataMap(Abso
Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126424329
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapFactory.java ---
@@ -0,0 +1,127 @@
+/*
+ * Licensed to the Apache Soft
Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/1099#discussion_r126438727
--- Diff:
core/src/main/java/org/apache/carbondata/core/indexstore/DataMapStoreManager.java
---
@@ -69,32 +72,54 @@ public TableDataMap getDataMap(A
Github user asfgit closed the pull request at:
https://github.com/apache/carbondata/pull/1099
---
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
33 matches
Mail list logo