Grant Henke has posted comments on this change. Change subject: [java] excluded slf4j from async dependency ......................................................................
Patch Set 2: I don't have any formal write up on version ranges advice. But essentially the advice is that though ranges may be useful during development, the released version used should be static. Since async uses the version range '[1.6.0,)' for slf4j, each time a build is run a new version of slf4j could be pulled in. If slf4j in the future makes a breaking change, there is no way to prevent it from being pulled into current or old builds. The maven docs on version ranges are here: http://maven.apache.org/enforcer/enforcer-rules/versionRanges.html -- To view, visit http://gerrit.cloudera.org:8080/7320 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia557729ef50b3736a53224bae64663652aeb0722 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: No
