[kudu-CR] [java] Sync gradle build with dependecy changes
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] Sync gradle build with dependecy changes .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/7645 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic4dec0486870d7851e4fe8f4034b820eaab48715 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant HenkeGerrit-Reviewer: Grant Henke Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-HasComments: No
[kudu-CR] [java] Sync gradle build with dependecy changes
Jean-Daniel Cryans has submitted this change and it was merged. Change subject: [java] Sync gradle build with dependecy changes .. [java] Sync gradle build with dependecy changes Adds the newly added parquet and spark-avro dependencies from commit 5d53a3b7146c6bcf330ddae036cfc42eeb7b7849 Change-Id: Ic4dec0486870d7851e4fe8f4034b820eaab48715 Reviewed-on: http://gerrit.cloudera.org:8080/7645 Reviewed-by: Jean-Daniel CryansTested-by: Jean-Daniel Cryans --- M java/gradle/dependencies.gradle M java/kudu-client-tools/build.gradle M java/kudu-spark-tools/build.gradle 3 files changed, 6 insertions(+), 0 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/7645 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic4dec0486870d7851e4fe8f4034b820eaab48715 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke Gerrit-Reviewer: Grant Henke Gerrit-Reviewer: Jean-Daniel Cryans
[kudu-CR] [java] Sync gradle build with dependecy changes
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] Sync gradle build with dependecy changes .. Patch Set 1: Code-Review+2 > I ran 'gradle clean assemble` to make sure this fixed compile > errors. The Jenkins failures appear to be from flaky tests. Yeah I was asking because I was pretty we didn't run gradle in the build so that's the only way we can verify the change. -- To view, visit http://gerrit.cloudera.org:8080/7645 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic4dec0486870d7851e4fe8f4034b820eaab48715 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant HenkeGerrit-Reviewer: Grant Henke Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No
[kudu-CR] [java] Sync gradle build with dependecy changes
Grant Henke has posted comments on this change. Change subject: [java] Sync gradle build with dependecy changes .. Patch Set 1: I ran 'gradle clean assemble` to make sure this fixed compile errors. The Jenkins failures appear to be from flaky tests. -- To view, visit http://gerrit.cloudera.org:8080/7645 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic4dec0486870d7851e4fe8f4034b820eaab48715 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant HenkeGerrit-Reviewer: Grant Henke Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: No
[kudu-CR] [java] Sync gradle build with dependecy changes
Jean-Daniel Cryans has posted comments on this change. Change subject: [java] Sync gradle build with dependecy changes .. Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/7645/1//COMMIT_MSG Commit Message: Line 11: 5d53a3b7146c6bcf330ddae036cfc42eeb7b7849 Did you try a full gradle build? -- To view, visit http://gerrit.cloudera.org:8080/7645 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ic4dec0486870d7851e4fe8f4034b820eaab48715 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant HenkeGerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
[kudu-CR] [java] Sync gradle build with dependecy changes
Grant Henke has uploaded a new change for review. http://gerrit.cloudera.org:8080/7645 Change subject: [java] Sync gradle build with dependecy changes .. [java] Sync gradle build with dependecy changes Adds the newly added parquet and spark-avro dependencies from commit 5d53a3b7146c6bcf330ddae036cfc42eeb7b7849 Change-Id: Ic4dec0486870d7851e4fe8f4034b820eaab48715 --- M java/gradle/dependencies.gradle M java/kudu-client-tools/build.gradle M java/kudu-spark-tools/build.gradle 3 files changed, 6 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/7645/1 -- To view, visit http://gerrit.cloudera.org:8080/7645 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic4dec0486870d7851e4fe8f4034b820eaab48715 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke