> On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > core/src/main/java/org/apache/oozie/action/hadoop/GitActionExecutor.java > > Lines 182-183 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067411#file2067411line182> > > > > These nested ifs can be merged.
Done. > On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > sharelib/git/src/main/java/org/apache/oozie/action/hadoop/GitMain.java > > Lines 200-201 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067428#file2067428line200> > > > > Do we need this? Removed. > On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > sharelib/git/src/main/java/org/apache/oozie/action/hadoop/GitOperations.java > > Lines 92 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067429#file2067429line92> > > > > Typo: "Checkiging" Fixed. > On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestGitActionExecutor.java > > Lines 114 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067431#file2067431line114> > > > > Better error message would be "Expected ActionExecutorException" Done. > On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestGitActionExecutor.java > > Lines 117 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067431#file2067431line117> > > > > Better error message would be "Unexpected exception message" or sth > > like that Done. > On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestGitActionExecutor.java > > Lines 148-154 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067431#file2067431line148> > > > > What are we testing here exactly? > > > > I can't see any assert() calls. Done. > On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestGitActionExecutor.java > > Lines 183-188 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067431#file2067431line183> > > > > Same here Done. > On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestGitMain.java > > Lines 101-104 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067432#file2067432line101> > > > > This should be replaced with a waitFor() method call - see Oozie > > codebase for examples. Done. > On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestIntegrationGitActionExecutor.java > > Lines 30 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067433#file2067433line30> > > > > Unused import Done. > On Aug. 7, 2018, 12:38 p.m., Peter Bacsko wrote: > > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestIntegrationGitActionExecutor.java > > Lines 86 (patched) > > <https://reviews.apache.org/r/59620/diff/10/?file=2067433#file2067433line86> > > > > readContent variable is unused Done. - András ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59620/#review206939 ----------------------------------------------------------- On Aug. 3, 2018, 10 p.m., Clay B. wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/59620/ > ----------------------------------------------------------- > > (Updated Aug. 3, 2018, 10 p.m.) > > > Review request for oozie and András Piros. > > > Bugs: OOZIE-2877 > https://issues.apache.org/jira/browse/OOZIE-2877 > > > Repository: oozie-git > > > Description > ------- > > OOZIE-2877 - Oozie Git Action > > > Diffs > ----- > > client/src/main/resources/git-action-1.0.xsd PRE-CREATION > core/src/main/java/org/apache/oozie/action/hadoop/GitActionExecutor.java > PRE-CREATION > core/src/main/resources/oozie-default.xml b69d2c9 > core/src/test/java/org/apache/oozie/test/XTestCase.java 661970d > docs/src/site/twiki/WorkflowFunctionalSpec.twiki 76cbe21 > examples/src/main/apps/git/job.properties PRE-CREATION > examples/src/main/apps/git/workflow.xml PRE-CREATION > fluent-job/fluent-job-api/pom.xml 4c9b853 > > fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/action/GitAction.java > PRE-CREATION > > fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/action/GitActionBuilder.java > PRE-CREATION > > fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/mapping/ExplicitNodeConverter.java > 7bb82e5 > > fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/mapping/GitConfigurationConverter.java > PRE-CREATION > > fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/mapping/GitPrepareConverter.java > PRE-CREATION > > fluent-job/fluent-job-api/src/main/java/org/apache/oozie/fluentjob/api/serialization/WorkflowMarshaller.java > ec56554 > fluent-job/fluent-job-api/src/main/resources/action_mappings.xml a5f890e > fluent-job/fluent-job-api/src/main/xjb/bindings.xml 48f6890 > pom.xml 92358aa > sharelib/git/pom.xml PRE-CREATION > sharelib/git/src/main/java/org/apache/oozie/action/hadoop/GitMain.java > PRE-CREATION > > sharelib/git/src/main/java/org/apache/oozie/action/hadoop/GitOperations.java > PRE-CREATION > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/GitServer.java > PRE-CREATION > > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestGitActionExecutor.java > PRE-CREATION > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestGitMain.java > PRE-CREATION > > sharelib/git/src/test/java/org/apache/oozie/action/hadoop/TestIntegrationGitActionExecutor.java > PRE-CREATION > sharelib/pom.xml 6a0864d > src/main/assemblies/sharelib.xml 07dc69c > webapp/pom.xml fd3f89f > > > Diff: https://reviews.apache.org/r/59620/diff/10/ > > > Testing > ------- > > Tested using unit and integration tests. Still need to: > * Test on a cluster > * Test with an authenticated SSH hosted Git repo > > Sumitted a request to the JGit community as their branch pulling code seems > to have an > [issue](https://dev.eclipse.org/mhonarc/lists/jgit-dev/msg03343.html). > > Andras has tested against his patch 011 with results at > https://issues.apache.org/jira/browse/OOZIE-2877?focusedCommentId=16459523&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16459523 > > > File Attachments > ---------------- > > 0001-OOZIE-2877-Oozie-Git-Action.patch > > https://reviews.apache.org/media/uploaded/files/2017/05/29/24f90a78-3dc1-49fe-bf29-5927a3cd5e72__0001-OOZIE-2877-Oozie-Git-Action.patch > Patch > > https://reviews.apache.org/media/uploaded/files/2017/05/29/dd23dd72-67e0-456f-9b52-e566d8d17d16__0001-OOZIE-2877-Oozie-Git-Action.patch > > > Thanks, > > Clay B. > >
