mikewalch commented on issue #726: log deprecated properties
URL: https://github.com/apache/accumulo/pull/726#issuecomment-434389181
 
 
   @mschnitzler,  Thanks for the PR!  Looks great but I ran into 2 minor issues 
when I tested it:
   * The warnings would print even when deprecated property was not set
   * The warnings would print multiple times as the validate() method in 
ConfigSanityCheck is called multiples times (like when a table is created).
   
   I created a pull request to your branch that should fix these issues.  Let 
me know what you think.  If you merge it, this pull request will be updated 
with my changes.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to