bq. Anything in test/ can't be public API by definition. Move it into src/ if that is desired.
If that has always been our stance, we should probably clarify it in our documentation. Or maybe it's already there and i missed it. Didn't find anything searching for 'test' in http://hbase.apache.org/book.html#hbase.versioning. I see IA.Public annotations on follow classes under **/test/ path: HBCommonTU, HBTU, MiniHBaseCluster, CodecPerformance, and HBaseZKTestingUtility At least MiniHBaseCluster seems like it might be useful to downstream projects. Others may have nuggets which might make sense in public api. Not sure what's best though - moving the classes + declaration that test/ code is private, or letting annotations do they work as they do work src/ code. -- Appy On Fri, Feb 2, 2018 at 5:56 PM, Andrew Purtell <apurt...@apache.org> wrote: > Phoenix is being a bit naughty. On the other hand, the status of this class > isn't clear. > > On Fri, Jan 26, 2018 at 2:10 PM, Ted Yu <yuzhih...@gmail.com> wrote: > > > At least Phoenix is using it. > > If you search in Phoenix repo, you would see a lot of classes with: > > > > import org.apache.hadoop.hbase.HBaseTestingUtility; > > > > If you have specific method(s) which you consider changing, you can ask > > dev@phoenix. > > > > FYI > > > > On Fri, Jan 26, 2018 at 2:01 PM, Mike Drob <md...@apache.org> wrote: > > > > > Can we make changes between releases? Do we know who else is using it? > > > > > > Mike > > > > > > > > > -- > Best regards, > Andrew > > Words like orphans lost among the crosstalk, meaning torn from truth's > decrepit hands > - A23, Crosstalk > -- -- Appy
