Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Drill >> mail # dev >> Review Request: Added JSONScanPOP and JSONBatchCreator


Copy link to this message
-
Review Request: Added JSONScanPOP and JSONBatchCreator

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12046/
-----------------------------------------------------------

Review request for drill and Jacques Nadeau.
Description
-------

Added JSONBatchCreator and JSONScanPOP. Also included some ValueVector fixes.
Diffs
-----

  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/config/JSONScanBatchCreator.java PRE-CREATION
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/config/JSONScanPOP.java PRE-CREATION
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ImplCreator.java d98c107
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/ScanBatch.java 33c1e29
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/materialize/VectorRecordMaterializer.java 7929296
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/record/RecordBatchLoader.java d990198
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/record/vector/AbstractFixedValueVector.java 8748285
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/record/vector/BaseValueVector.java 69cd628
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/record/vector/Bit.java 533e3bd
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/record/vector/NullableValueVector.java 8c3d7f6
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/record/vector/VariableVector.java dc8fec4
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/schema/DiffSchema.java b654a92
  sandbox/prototype/exec/java-exec/src/main/java/org/apache/drill/exec/store/JSONRecordReader.java 42d1be5
  sandbox/prototype/exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/TestSimpleFragmentRun.java 594a3a2
  sandbox/prototype/exec/java-exec/src/test/resources/physical_json_scan_test1.json PRE-CREATION

Diff: https://reviews.apache.org/r/12046/diff/
Testing
-------
Thanks,

Timothy Chen