accumulo-dev mailing list archives

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

    https://github.com/apache/accumulo/pull/119
  
    I'm not sure why or when the "true" parameter should be used in the cases where it currently
is used (in ClusterUser, randomwalk Environment, and the cluster harness classes). They don't
appear to be tests of the method, but rather, tests which just happen to use that method.
So, the tests themselves won't be deprecated with this method's deprecation.
    
    Since I'm not familiar enough to know how to move those tests off of this method, I'm
just going to make sure we don't create additional warnings, by suppressing the use of deprecation
warning in those areas where this method is still used.


---
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