Review Board 1.7.22


DRILL-45: Add builders for PlanProperties, LogicalPlan, Store and Scan

Review Request #11783 - Created June 10, 2013 and updated

Ben Becker
Reviewers
drill-git
jacques
drill-git
Looking for validation of builder apprach.
 - Includes LogicalPlanBuilder and PlanPropertiesBuilder attached to JIRA-45; rebased to latest execwork.
 - Includes ScanBuilder and StoreBuilder as well.

Note: One of the original builders also had a 'toBuilder()' method.  Is that something we should add for all builders?

 
Total:
1
Open:
0
Resolved:
1
Dropped:
0
Status:
From:
Review request changed
Updated (Oct. 30, 2013, 11:16 p.m.)
Rebased onto latest master.

GitHub branch: https://github.com/vrtx/incubator-drill/tree/DRILL-45-commit
JIRA: https://issues.apache.org/jira/browse/DRILL-45 (latest 'DRILL-45.patch' attachment)