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




sharelib/hcatalog/src/main/java/org/apache/oozie/util/HCatURI.java (line 86)
<https://reviews.apache.org/r/52519/#comment219889>

    If we create he LinkedHashMap at declaration, there's no need to do it here.


Could you update the documentation here?
https://oozie.apache.org/docs/4.2.0/WorkflowFunctionalSpec.html#a3.2.2.5_Syntax

- Peter Cseh


On Oct. 4, 2016, 5:47 p.m., Abhishek Bafna wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/52519/
> -----------------------------------------------------------
> 
> (Updated Oct. 4, 2016, 5:47 p.m.)
> 
> 
> Review request for oozie.
> 
> 
> Bugs: OOZIE-2393
>     https://issues.apache.org/jira/browse/OOZIE-2393
> 
> 
> Repository: oozie-git
> 
> 
> Description
> -------
> 
> Allow table drop in hcat prepare
> 
> 
> Diffs
> -----
> 
>   core/src/main/java/org/apache/oozie/dependency/HCatURIHandler.java 67b37ec 
>   core/src/test/java/org/apache/oozie/action/hadoop/TestFsActionExecutor.java 
> 5345ae6 
>   
> core/src/test/java/org/apache/oozie/action/hadoop/TestLauncherHCatURIHandler.java
>  66d4ecc 
>   core/src/test/java/org/apache/oozie/dependency/TestHCatURIHandler.java 
> a49eba5 
>   core/src/test/java/org/apache/oozie/test/MiniHCatServer.java 8699ff8 
>   core/src/test/java/org/apache/oozie/test/XHCatTestCase.java 2adbee7 
>   
> sharelib/hcatalog/src/main/java/org/apache/oozie/action/hadoop/HCatLauncherURIHandler.java
>  6798768 
>   sharelib/hcatalog/src/main/java/org/apache/oozie/util/HCatURI.java 8ec3fae 
> 
> Diff: https://reviews.apache.org/r/52519/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Abhishek Bafna
> 
>

Reply via email to