Grant Henke has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10424
Change subject: [java] Remove private method usage in kudu-mapreduce ...................................................................... [java] Remove private method usage in kudu-mapreduce kudu-mapreduce uses private APIs from KuduPredicate. This is an issue because the API returns a protobuf generated class and uses shaded APIs. This isn’t strictly an issue in the gradle build itself because calling the shaded methods is ok, but it is an issue in IDEs like Intellij. Regardless a shaded dependency is not part of the public API and should not be exposed/used because it can cause problems like this. A follow on patch will work on enforcing that protobuf is not exposed in the client API so this doesn’t happen again. Change-Id: I7ce54f715f970d1c7f9433e39af27d618e73fe6a --- M java/kudu-client/src/main/java/org/apache/kudu/client/KuduPredicate.java M java/kudu-mapreduce/src/main/java/org/apache/kudu/mapreduce/KuduTableInputFormat.java M java/kudu-mapreduce/src/main/java/org/apache/kudu/mapreduce/KuduTableMapReduceUtil.java 3 files changed, 38 insertions(+), 12 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/24/10424/1 -- To view, visit http://gerrit.cloudera.org:8080/10424 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I7ce54f715f970d1c7f9433e39af27d618e73fe6a Gerrit-Change-Number: 10424 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <granthe...@apache.org>
