HBASE-15876 is the removal (with a subissue for the late deprecation). I can undo if premature, if objection. Thanks, St.Ack
On Mon, May 23, 2016 at 8:04 AM, Stack <[email protected]> wrote: > Thanks lads for the feedback. > St.Ack > > On Sun, May 22, 2016 at 10:55 PM, Enis Söztutar <[email protected]> > wrote: > >> It sounds good. HTable is already deprecated, so removing a method with >> that arg should be fine. We can deprecate it still in 1.3 and later just >> in >> case. >> >> Enis >> >> On Sun, May 22, 2016 at 8:01 PM, Heng Chen <[email protected]> >> wrote: >> >> > +1. >> > >> > 2016-05-23 1:29 GMT+08:00 Stack <[email protected]>: >> > >> > > HBASE-15875 wants to remove HTable and HTableInterface, classes that >> have >> > > been deprecated since before hbase-1.0.0 release. Unfortunately, we >> > missed >> > > deprecating the method doBulkLoad(Path hfofDir, final HTable table) in >> > > LoadIncrementalHFiles, a public, stable class. >> > > >> > > My thinking is that its removal is 'ok' -- with proper notice -- since >> > this >> > > a method used by offline tooling. My thinking is that as long as >> proper >> > > notice, folks can prepare their tooling ahead of time changing them to >> > use >> > > the alternative that takes a Table instance in time for the upgrade. >> This >> > > method is also 'damaged' given it has a param that was properly >> > deprecated >> > > -- i.e. the HTable instance -- but nonetheless, its removal will be a >> > > breaking change in 2.0. >> > > >> > > You lot all good w/ this? >> > > Thanks, >> > > St. >> > > >> > >> > >
