> On Aug. 8, 2014, 8:33 a.m., Purshotam Shah wrote: > > core/src/main/java/org/apache/oozie/CoordinatorEngine.java, line 37 > > <https://reviews.apache.org/r/24299/diff/1/?file=651739#file651739line37> > > > > Why Hbase util? This means that Oozie always need have hbase-*.jar, > > which might cause other conflict. > > shwethags wrote: > Pair is defined in a lot of dependent jars. So, wanted to re-use instead > of defining our own. If we remove hbase dependency, we can switch to others > or define our own. Looks ok for now > > Rohini Palaniswamy wrote: > Ah. Did not notice that earlier. This needs to be changed. We cannot > introduce a dependency on hbase. HBase is supposedly pluggable.
Why does oozie depend on hbase by the way? - shwethags ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24299/#review50019 ----------------------------------------------------------- On Aug. 8, 2014, 9:24 a.m., shwethags wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/24299/ > ----------------------------------------------------------- > > (Updated Aug. 8, 2014, 9:24 a.m.) > > > Review request for oozie. > > > Bugs: OOZIE-1950 > https://issues.apache.org/jira/browse/OOZIE-1950 > > > Repository: oozie-git > > > Description > ------- > > Adds filter on nominal time in get coord actions > > > Diffs > ----- > > client/src/main/java/org/apache/oozie/client/OozieClient.java b468186 > core/src/main/java/org/apache/oozie/CoordinatorEngine.java dd5c703 > core/src/main/java/org/apache/oozie/command/coord/CoordJobXCommand.java > 5eaf062 > > core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsSubsetJPAExecutor.java > a531798 > core/src/test/java/org/apache/oozie/TestCoordinatorEngineSimple.java > 1904519 > > core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsSubsetJPAExecutor.java > 147bdb7 > docs/src/site/twiki/DG_CommandLineTool.twiki 4a07711 > > Diff: https://reviews.apache.org/r/24299/diff/ > > > Testing > ------- > > UTs > > > Thanks, > > shwethags > >
