Ah ok, perfect! That was the reason why I removed it :) On Mon, Nov 3, 2014 at 9:10 AM, Stephan Ewen <[email protected]> wrote:
> You do not really need a HBase data sink. You can call "DataSet.output(new > HBaseOutputFormat())" > > Stephan > Am 02.11.2014 23:05 schrieb "Flavio Pompermaier" <[email protected]>: > > > Just one last thing..I removed the HbaseDataSink because I think it was > > using the old APIs..can someone help me in updating that class? > > > > On Sun, Nov 2, 2014 at 10:55 AM, Flavio Pompermaier < > [email protected]> > > wrote: > > > > > Indeed this time the build has been successful :) > > > > > > On Sun, Nov 2, 2014 at 10:29 AM, Fabian Hueske <[email protected]> > > wrote: > > > > > >> You can also setup Travis to build your own Github repositories by > > linking > > >> it to your Github account. That way Travis can build all your branches > > >> (and > > >> you can also trigger rebuilds if something fails). > > >> Not sure if we can manually trigger retrigger builds on the Apache > > >> repository. > > >> > > >> Support for Hadoop 1 and 2 is indeed a very good addition :-) > > >> > > >> For the discusion about the PR itself, I would need a bit more time to > > >> become more familiar with HBase. I do also not have a HBase setup > > >> available > > >> here. > > >> Maybe somebody else of the community who was involved with a previous > > >> version of the HBase connector could comment on your question. > > >> > > >> Best, Fabian > > >> > > >> 2014-11-02 9:57 GMT+01:00 Flavio Pompermaier <[email protected]>: > > >> > > >> > As suggestes by Fabian I moved the discussion on this mailing list. > > >> > > > >> > I think that what is still to be discussed is how to retrigger the > > >> build > > >> > on Travis (I don't have an account) and if the PR can be integrated. > > >> > > > >> > Maybe what I can do is to move the HBase example in the test package > > >> (right > > >> > now I left it in the main folder) so it will force Travis to > rebuild. > > >> > I'll do it within a couple of hours. > > >> > > > >> > Another thing I forgot to say is that the hbase extension is now > > >> compatible > > >> > with both hadoop 1 and 2. > > >> > > > >> > Best, > > >> > Flavio > > >> > > > > > >
