[kudu-CR] java tests: Also search for build bindir from cwd

2016-12-03 Thread Mike Percy (Code Review)
Mike Percy has abandoned this change. Change subject: java tests: Also search for build bindir from cwd .. Abandoned OK, I'll abandon since we have a workaround. I'll post a separate CR to document this inline so we don't keep

[kudu-CR] java tests: Also search for build bindir from cwd

2016-12-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: java tests: Also search for build bindir from cwd .. Patch Set 1: No, the concern is that developers will unwittingly run tests against a version of kudu-client i

[kudu-CR] java tests: Also search for build bindir from cwd

2016-12-02 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: java tests: Also search for build bindir from cwd .. Patch Set 1: Is the concern w/ this patch that we will somehow merge something broken? If so, doesn't jenkins

[kudu-CR] java tests: Also search for build bindir from cwd

2016-12-02 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. Change subject: java tests: Also search for build bindir from cwd .. Patch Set 1: This patch allows the following pattern: $ mvn clean install -pl kudu-flume-sink -am -DskipTests

[kudu-CR] java tests: Also search for build bindir from cwd

2016-12-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: java tests: Also search for build bindir from cwd .. Patch Set 1: I've previously rejected this patch, because I think the premise of running tests against an ins

[kudu-CR] java tests: Also search for build bindir from cwd

2016-12-02 Thread Mike Percy (Code Review)
Hello Dan Burkert, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5328 to review the following change. Change subject: java tests: Also search for build bindir from cwd .. java tests: Also