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

(Updated July 27, 2017, 1:20 p.m.)


Review request for oozie, Attila Sasvari, Peter Cseh, Peter Bacsko, and Robert 
Kanter.


Changes
-------

Using commons-lang `StringUtils` instead of Guava `Strings`.


Repository: oozie-git


Description
-------

Parse `spark-defaults.conf` values with spaces without needing the quotes. That 
is, quotes should not be mandatory but optional to retain backwards 
compatibility.


Diffs (updated)
-----

  docs/src/site/twiki/DG_SparkActionExtension.twiki 
863bd89cf576d2fb7be76c61ca139218c33a5f66 
  
sharelib/spark/src/main/java/org/apache/oozie/action/hadoop/SparkOptionsSplitter.java
 30def6f05b7a17b8ce66ed766a9d64c171796ac5 
  
sharelib/spark/src/test/java/org/apache/oozie/action/hadoop/TestSparkArgsExtractor.java
 7db26a6b81083d7664bef8ff14c8e14df4572fc1 
  
sharelib/spark/src/test/java/org/apache/oozie/action/hadoop/TestSparkOptionsSplitter.java
 02786a2476fc981af77688a22aa9c3a8fc0cce2a 


Diff: https://reviews.apache.org/r/60963/diff/5/

Changes: https://reviews.apache.org/r/60963/diff/4-5/


Testing
-------

Following tests are run:

* `TestSparkOptionsSplitter`
* `TestSparkArgsExtractor`
* `TestSparkMain`


Thanks,

András Piros

Reply via email to