This is an automatically generated e-mail. To reply, visit:

(Updated Oct. 12, 2016, 2:45 p.m.)

Review request for oozie and Robert Kanter.


Small refactor.

Repository: oozie-git


We go for that solution:

1. create a new script as `bin/test-patch-11-findbugs-diff` so that it comes 
right after compile

2. `mvn clean verify -DskipTests` before and after applying the patch, and 
saving each and every module's `target/findbugs/findbugsXml.xml` to a secure 

3. call the `findbugs-diff` app for all the before / after FindBugs xml pairs

4. analyze all the `findbugs-new.xml` entries for nodes with XPath 
`/BugCollection/BugInstance` entries. Stop on any BugInstance w/ `priority` and 
/ or `rank` lower than threshold

5. have the reports merged and be downloadable in any case in following 

* -1 for bugs that are too serious to apply the patch with them
* 0 for other new bugs
* +1 if there is no new bug

Diffs (updated)

  bin/test-patch-11-findbugs-diff PRE-CREATION 
  bin/test-patch-11-findbugs-diff-0.1.0-all.jar.md5 PRE-CREATION 
  pom.xml 83a1d42 

Diff: https://reviews.apache.org/r/52429/diff/



András Piros

Reply via email to