accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From joshelser <...@git.apache.org>
Subject [GitHub] accumulo issue #119: ACCUMULO-4348 Deprecate KerberosToken constructor with ...
Date Tue, 21 Jun 2016 21:48:02 GMT
Github user joshelser commented on the issue:

    https://github.com/apache/accumulo/pull/119
  
    > Replication uses that constructor, but could be replaced with the one I added as
it sets the flag to false. Uses that set the flag to true are all in ITs or tests.
    
    Ok, thanks!
    
    > I think it's probably best if our code stops using the deprecated method, and any
tests for the deprecated method itself, should themselves be deprecated.
    
    +1 making sure we use the methods/constructors we expect users to use is a great test
that things work as we want.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message