[ 
https://issues.apache.org/jira/browse/OOZIE-1200?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13574909#comment-13574909
 ] 

Alejandro Abdelnur commented on OOZIE-1200:
-------------------------------------------

ok on the 2 APIs. On the URIContext, well, this is meant for the URIHandler 
only at the moment. Moving it inside, we could rename it to Context (as its 
outerclass would define its scope), if it stays as standalone class it should 
be renamed to URIHandlerContext as the current name URIContext seems to imply 
it is for URIs, which is not.
                
> Review of HCAT integration branch
> ---------------------------------
>
>                 Key: OOZIE-1200
>                 URL: https://issues.apache.org/jira/browse/OOZIE-1200
>             Project: Oozie
>          Issue Type: Sub-task
>          Components: coordinator, docs
>            Reporter: Alejandro Abdelnur
>             Fix For: trunk
>
>         Attachments: HCat-commandcoord-package-review.patch, 
> HCat-depedency-cache-review.patch, HCat-dependency-package-review.patch, 
> Hcat-service-util-tools-action-jms-packages-review-1.patch, 
> Hcat-service-util-tools-action-jms-packages-review.patch, 
> HCatSpec-comments.patch
>
>
> Given the number of commits (and fixes) that when into this branch, I think 
> the best way to review it is by posting patches with comments through out the 
> code. Then we can follow up the discussion as comments in this JIRA.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to