[kudu-CR] Running kudu-spark tests alone fail finding build dir

2016-10-05 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has abandoned this change. Change subject: Running kudu-spark tests alone fail finding build dir .. Abandoned After some back and forth offline, best to just leave this alone. The way to specify kudu binary di

[kudu-CR] kudu-spark pom.xml has malformed artifactId

2016-10-05 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has abandoned this change. Change subject: kudu-spark pom.xml has malformed artifactId .. Abandoned Taking Dan's advice and leaving this in even with the warning, since its how the community typically does it.

[kudu-CR] kudu-spark pom.xml has malformed artifactId

2016-10-05 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has posted comments on this change. Change subject: kudu-spark pom.xml has malformed artifactId .. Patch Set 1: I see. Annoying, but I suppose it'll save changing it in two places in the pom file. -- To view

[kudu-CR] Running kudu-spark tests alone fail finding build dir

2016-10-05 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has posted comments on this change. Change subject: Running kudu-spark tests alone fail finding build dir .. Patch Set 2: It seems a little 'hackish' anyway looking for the base kudu-client test class's locati

[kudu-CR] Running kudu-spark tests alone fail finding build dir

2016-10-05 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has posted comments on this change. Change subject: Running kudu-spark tests alone fail finding build dir .. Patch Set 2: So this is interesting. If I go under java/kudu-spark and from there run "mvn clean pac

[kudu-CR] Running kudu-spark tests alone fail finding build dir

2016-10-05 Thread Mladen Kovacevic (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4630 to look at the new patch set (#2). Change subject: Running kudu-spark tests alone fail finding build dir .. Running

[kudu-CR] Running kudu-spark tests alone fail finding build dir

2016-10-05 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has uploaded a new change for review. http://gerrit.cloudera.org:8080/4630 Change subject: Running kudu-spark tests alone fail finding build dir .. Running kudu-spark tests alone fail finding build dir When r

[kudu-CR] kudu-spark pom.xml has malformed artifactId

2016-10-04 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has uploaded a new change for review. http://gerrit.cloudera.org:8080/4628 Change subject: kudu-spark pom.xml has malformed artifactId .. kudu-spark pom.xml has malformed artifactId The pom.xml file in the ku

[kudu-CR] Replace incubator-kudu links with kudu in docs

2016-08-24 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has posted comments on this change. Change subject: Replace incubator-kudu links with kudu in docs .. Patch Set 4: (3 comments) Alex and Will, thanks for the comments. Let me know if anything else is needed h

[kudu-CR] Replace incubator-kudu links with kudu in docs

2016-08-24 Thread Mladen Kovacevic (Code Review)
Hello Will Berkeley, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4107 to look at the new patch set (#4). Change subject: Replace incubator-kudu links with kudu in docs ..

[kudu-CR] Update contributing doc page with apache/kudu instead of apache/incubator-kudu

2016-08-23 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has posted comments on this change. Change subject: Update contributing doc page with apache/kudu instead of apache/incubator-kudu .. Patch Set 3: > There's a bunch of similar git clone links that use incubato

[kudu-CR] Update contributing doc page with apache/kudu instead of apache/incubator-kudu

2016-08-23 Thread Mladen Kovacevic (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4107 to look at the new patch set (#3). Change subject: Update contributing doc page with apache/kudu instead of apache/incubator-kudu .

[kudu-CR] Update contributing doc page with apache/kudu instead of apache/incubator-kudu

2016-08-23 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has uploaded a new patch set (#2). Change subject: Update contributing doc page with apache/kudu instead of apache/incubator-kudu .. Update contributing doc page with apache/kudu instead of apache/incubator-kudu

[kudu-CR] Update contributing doc page with apache/kudu instead of apache/incubator-kudu

2016-08-23 Thread Mladen Kovacevic (Code Review)
Mladen Kovacevic has uploaded a new change for review. http://gerrit.cloudera.org:8080/4107 Change subject: Update contributing doc page with apache/kudu instead of apache/incubator-kudu .. Update contributing doc page with a