Github user joshelser commented on the issue:

    https://github.com/apache/accumulo/pull/242
  
    > I changed to use the public variables at the suggestion of @ctubbsii, but 
we could discuss the best approach and change it to that.
    
    Ok. Mostly I was thinking that if we are moving these _towards_ being 
publicly consumable, it would be better if we encapsulate the state and just 
expose it via methods (that gives us more flexibility).
    
    > I tested on both FF and Chrome, I couldn't get the proxy to work for IE 
so I haven't been able to test it.
    
    Ok, this would be good to keep in mind before merging. IE still has a bit 
of prevalence in "the enterprise". We should perform some diligence before 
merge to make sure that we aren't shipping something entirely busted on the 
major IE versions.


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to