bq. Since it kind of serves a purpose, I'll leave the developer list as-is if others are not in favor of removing it.
Sure, sounds good to me. Here's some more feedback: Would it be possible to only deprecate the old name "cask.*" property names by looking for the new one first and falling back to the old one if not found? That way, existing clients will continue to work when the server is upgraded. https://git-wip-us.apache.org/repos/asf?p=incubator-tephra.git;a=blobdiff;f=tephra-core/src/main/java/org/apache/tephra/TxConstants.java;h=d9624867c5f30c63ba9a1790fd70030196f3ca9a;hp=7f7fc8c47c6589509eac6145c3eebad3542d2ede;hb=d038cb300ac603ae103eb1ae060aa84e3e9d7eb6;hpb=26ca8e1ac8f99b902fb9effef4faebe174543ff1 Thanks for the quick work on this, Poorna. Everything else looks great. James On Mon, May 9, 2016 at 1:18 PM, Poorna Chandra <[email protected]> wrote: > > > > FWIW, HBase has the developer list in the pom. This makes for a nice > flow > > when a new committer is added -- the first task for the new committer is > to > > add themselves to the developer list. This makes sure that commit karma > is > > working right away and also makes the list self-maintaining. > > > > > But if others want to drop it here, that is fine by me as well. > > > Since it kind of serves a purpose, I'll leave the developer list as-is if > others are > not in favor of removing it. > > Poorna. > > On Mon, May 9, 2016 at 12:43 PM, Gary Helmling <[email protected]> > wrote: > > > > > > > > - I've never seen the developers listed in the pom - that seems hard > to > > > > maintain. IMHO, I'd just remove it entirely. > > > > > > > > > > Makes sense, will remove it. > > > > > > > > > > > FWIW, HBase has the developer list in the pom. This makes for a nice > flow > > when a new committer is added -- the first task for the new committer is > to > > add themselves to the developer list. This makes sure that commit karma > is > > working right away and also makes the list self-maintaining. > > > > But if others want to drop it here, that is fine by me as well. > > >
