That is one class I removed because it was using the deprecated API GenericDataSink..I can restore them but the it will be a good idea to remove those warning (also because from what I understood the Record APIs are going to be removed).
On Mon, Nov 3, 2014 at 9:51 AM, Fabian Hueske <[email protected]> wrote: > I'm not familiar with the HBase connector code, but are you maybe looking > for the GenericTableOutputFormat? > > 2014-11-03 9:44 GMT+01:00 Flavio Pompermaier <[email protected]>: > > > | was trying to modify the example setting hbaseDs.output(new > > HBaseOutputFormat()); but I can't see any HBaseOutputFormat class..maybe > we > > shall use another class? > > > > On Mon, Nov 3, 2014 at 9:39 AM, Flavio Pompermaier <[email protected] > > > > wrote: > > > > > Maybe that's something I could add to the HBase example and that could > be > > > better documented in the Wiki. > > > > > > Since we're talking about the wiki..I was looking at the Java API ( > > > > > > http://flink.incubator.apache.org/docs/0.6-incubating/java_api_guide.html) > > > and the link to the KMeans example is not working (where it says For a > > > complete example program, have a look at KMeans Algorithm). > > > > > > Best, > > > Flavio > > > > > > > > > On Mon, Nov 3, 2014 at 9:12 AM, Flavio Pompermaier < > [email protected] > > > > > > wrote: > > > > > >> 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 > > >>> > >> > > >>> > > > > >>> > > > >>> > > >> > > > > > >
