-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31107/
-----------------------------------------------------------
(Updated March 6, 2015, 2:59 p.m.)
Review request for drill, Chris Westin, Jacques Nadeau, Steven Phillips, and
Venki Korukanti.
Changes
-------
Added Unit tests
Added number of threads as a metric to PartitionSender operator
Fixed partitioners distribution algorithm to do even distribution between
threads
Bugs: DRILL-2210
https://issues.apache.org/jira/browse/DRILL-2210
Repository: drill-git
Description
-------
In addition to description
Fixed few classes that did not handle multithreading well
Added/Changed some Stats behavior to allow stats merge from multiple threads,
since again this class is not suitable to be used in multithreaded environment
Introduced new decorator class to handle multi thrteading (or not) to minimize
changes to ParitionSenderRootExec class
Diffs (updated)
-----
exec/java-exec/src/main/java/org/apache/drill/exec/compile/CodeCompiler.java
7cc350e
exec/java-exec/src/main/java/org/apache/drill/exec/ops/FragmentContext.java
e413921
exec/java-exec/src/main/java/org/apache/drill/exec/ops/OperatorStats.java
0e9da0e
exec/java-exec/src/main/java/org/apache/drill/exec/physical/config/IteratorValidator.java
64cf7c5
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/SendingAccountor.java
7af7b65
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionSenderRootExec.java
a23bd7a
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/Partitioner.java
5ed9c39
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionerDecorator.java
PRE-CREATION
exec/java-exec/src/main/java/org/apache/drill/exec/physical/impl/partitionsender/PartitionerTemplate.java
71ffd41
exec/java-exec/src/main/java/org/apache/drill/exec/planner/fragment/Materializer.java
961b603
exec/java-exec/src/main/java/org/apache/drill/exec/planner/physical/PlannerSettings.java
bbfbbcb
exec/java-exec/src/main/java/org/apache/drill/exec/server/DrillbitContext.java
0fb10ff
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java
3d3e96f
exec/java-exec/src/main/java/org/apache/drill/exec/work/WorkManager.java
99c6ab8
exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/TestOptiqPlans.java
4aaaa78
exec/java-exec/src/test/java/org/apache/drill/exec/physical/impl/partitionsender/TestPartitionSender.java
PRE-CREATION
Diff: https://reviews.apache.org/r/31107/diff/
Testing
-------
Still need to provide Unit Tests.
Functional tests are passing
Performance tests were run and look promising for some queries
Thanks,
Yuliya Feldman
