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

(Updated April 14, 2015, 5:32 p.m.)


Review request for drill and Jacques Nadeau.


Changes
-------

Please review [diff/2/](https://reviews.apache.org/r/33177/diff/2/) (and not 
[diff/1/](https://reviews.apache.org/r/33177/diff/1/) or 
[diff/1-2/](https://reviews.apache.org/r/33177/diff/1-2/))

+ Based on top of master and 
[DRILL-2766](https://issues.apache.org/jira/browse/DRILL-2766)
+ First uploaded a spacing change to README.md for review board to accept 
(because DRILL-2766 is not merged into master yet)


Repository: drill-git


Description (updated)
-------

[DRILL-2508](https://issues.apache.org/jira/browse/DRILL-2508): Added a wrapper 
class for OptionValue to include status
Option status: BOOT, DEFAULT, CHANGED


Diffs (updated)
-----

  
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/FallbackOptionManager.java
 45d393c 
  
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/OptionManager.java
 4ffe9a3 
  
exec/java-exec/src/main/java/org/apache/drill/exec/server/options/SystemOptionManager.java
 1a8559e 
  exec/java-exec/src/main/java/org/apache/drill/exec/store/sys/SystemTable.java 
2c338ca 

Diff: https://reviews.apache.org/r/33177/diff/


Testing
-------


Thanks,

Sudheesh Katkam

Reply via email to