----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/8946/#review15401 -----------------------------------------------------------
Ship it! looks fine. no minor nitpicks - Mona Chitnis On Jan. 14, 2013, 11:25 p.m., Rohini Palaniswamy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/8946/ > ----------------------------------------------------------- > > (Updated Jan. 14, 2013, 11:25 p.m.) > > > Review request for oozie. > > > Description > ------- > > hcat:exists EL function and documentation updates. > > Also removed the enum from PrepareActions as it gives a CNFE > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/action/hadoop/PrepareActionsDriver.java > 1432982 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/java/org/apache/oozie/coord/HCatELFunctions.java > 1432982 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/main/resources/oozie-default.xml > 1432982 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/core/src/test/java/org/apache/oozie/coord/TestHCatELFunctions.java > 1432982 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/docs/src/site/twiki/CoordinatorFunctionalSpec.twiki > 1432982 > > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/docs/src/site/twiki/WorkflowFunctionalSpec.twiki > 1432982 > http://svn.apache.org/repos/asf/oozie/branches/hcat-intre/pom.xml 1432982 > > Diff: https://reviews.apache.org/r/8946/diff/ > > > Testing > ------- > > > Thanks, > > Rohini Palaniswamy > >
