You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2018/01/12 04:19:06 UTC

[GitHub] ctubbsii opened a new pull request #357: ACCUMULO-4611 Deprecate commons config in api

ctubbsii opened a new pull request #357: ACCUMULO-4611 Deprecate commons config in api
URL: https://github.com/apache/accumulo/pull/357
 
 
   In 3 commits:
   
   # Bump to version 1.9.0 as required by semver for API additions, and so we can deprecate and remove the offending API in 2.0.0 as discussed on the mailing list
   # Update apilyzer rules to find offending APIs with respect to commons-config
   # Fix
   
   The third "fix" commit is the most interesting one. If necessary, I can push the first two to a branch, and then then do a pull request with just the last commit against that branch, so it's easier to review on its own. Reviewers, let me know if you want me to do that.
   
   @joshelser Tagging you here for review, b/c GitHub won't show your GitHub user name in the "Request reviewers" dialog. Probably b/c you haven't sync'd with GitBox to be added to the repo? Not sure.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services