Dan Burkert has posted comments on this change.
Change subject: [java] Replace interface-annotations with Apache Yetus
annotations
......................................................................
Patch Set 8:
> The reason I said that is that the interface-annotations jar ends up packaged
> with Impala in CDH. Maybe it's a stray, though.
Right, but when they do another build of impala and point it to the new version
of Kudu, maven will automatically pull in yet annotations instead of the old
interface annotations jar, so it shouldn't require a change.
> That actually seems like binary compat to me, with the compat breakage being
> something different that I don't have a name for ("dependency switcheroo"?)
Yah, it's a somewhat pedantic definition of binary compat, which is why I don't
think we should tie ourselves to it.
--
To view, visit http://gerrit.cloudera.org:8080/7235
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I860e59f497d96f507d5a2b0ea517f9f8d0794dac
Gerrit-PatchSet: 8
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Will Berkeley <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-HasComments: No
