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

Ship it!


Ship It!

- Chris Westin


On March 17, 2015, 8:51 a.m., Sudheesh Katkam wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31938/
> -----------------------------------------------------------
> 
> (Updated March 17, 2015, 8:51 a.m.)
> 
> 
> Review request for drill and Venki Korukanti.
> 
> 
> Repository: drill-git
> 
> 
> Description
> -------
> 
> DRILL-2275: Added support to allow for querying cluster state information.
> + If table isDistributed(), BatchCreator and SystemTableScan allow for a 
> distributed query.
> + SystemRecordReader reads SystemRecords.
> + There is now a generic data type for static tables.
> + GroupScan can enforce width to be maximum width on ExcessiveExchangeRemover.
> + GroupScan has minimum width for SimpleParallelizer.
> 
> 
> Diffs
> -----
> 
>   common/src/main/java/org/apache/drill/common/JSONOptions.java 64e6d52 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/AbstractGroupScan.java
>  276ecb5 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/physical/base/GroupScan.java
>  23860a3 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/ParallelizationInfo.java
>  75a009e 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/SimpleParallelizer.java
>  f8d1803 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/Stats.java
>  e61b38f 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/StatsCollector.java
>  1f56556 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/visitor/ExcessiveExchangeIdentifier.java
>  a237014 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/RecordDataType.java 
> PRE-CREATION 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/pojo/PojoDataType.java
>  c1e64e6 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/MemoryRecord.java
>  PRE-CREATION 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/StaticDrillTable.java
>  c1e8dd1 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/SystemRecord.java
>  PRE-CREATION 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/SystemRecordReader.java
>  PRE-CREATION 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/SystemTable.java 
> 0bf2156 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/SystemTableBatchCreator.java
>  a1bec1e 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/SystemTablePlugin.java
>  2c70fd4 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/SystemTablePluginConfig.java
>  93fe68e 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/SystemTableScan.java
>  cdd0d18 
>   
> exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/ThreadsRecord.java
>  PRE-CREATION 
>   
> exec/java-exec/src/test/java/org/apache/drill/exec/store/sys/TestSystemTable.java
>  c1803bc 
> 
> Diff: https://reviews.apache.org/r/31938/diff/
> 
> 
> Testing
> -------
> 
> Tested on 3-node cluster and in embedded mode.
> ```
> > select * from sys.memory;
> +------------+--------------+------------+---------------------+
> | host_name  | user_port  | total_memory | heap_size  | direct_alloc_memory |
> +------------+--------------+------------+---------------------+
> | perfnode206.perf.lab | 31010      | 1073741824   | 395823432  | 5000000     
>         |
> | perfnode208.perf.lab | 31010      | 1073741824   | 337127496  | 2000000     
>         |
> | perfnode207.perf.lab | 31010      | 1073741824   | 289272760  | 2000000     
>         |
> +------------+--------------+------------+---------------------+
> ```
> 
> 
> Thanks,
> 
> Sudheesh Katkam
> 
>

Reply via email to