I would be happy to clean up and make the reflection-based Hammer available to those who desire it for current Accumulo versions. Any suggestions on the best/proper way to do this for the community? I assume it is bad practice to use the org.apache.accumulo package for code that isn't actually part of Accumulo, right?
On Thu, Jan 2, 2014 at 6:15 PM, Christopher <[email protected]> wrote: > On Thu, Jan 2, 2014 at 2:42 PM, Josh Elser <[email protected]> wrote: > > > > > > On 1/2/14, 2:16 PM, Keith Turner wrote: > >> > >> On Thu, Jan 2, 2014 at 1:51 PM, Christopher <[email protected]> > wrote: > >> > >>> I agree with Keith. Thanks for summarizing, Sean. > >>> > >>> I also favor option #2 for all existing versions, up through 1.6.0. > >>> > >> > >> What are your thoughts on reverting close() in 1.7.0-SNAPSHOT? > > > > > > IMO, I think a full revert makes sense, and then we can fully hash out > the > > new API without influence of what was already there. If we end up > > "re-adding" the changes, so be it, but it seems very unlikely that we'd > end > > up with the same final approach. > [snip] > > +1 >
