Alexey Serbin has posted comments on this change.

Change subject: [twitter-demo] use AUTO_FLUSH_BACKGROUND session
......................................................................


Patch Set 5:

> have you tried to run the demo app now?

Yes, I ran those before submitting the previous patch on MacOS X.  I tried both 
the demo:

./bin/ingest_firehose  -twitter_rpc_master_address=localhost:8765 
-twitter_firehose_source=file -twitter_firehose_file=./bin/example-tweets.txt

and the tests:

./bin/parser-test
./bin/oauth-test 

What else do you think I should do for better verification?

-- 
To view, visit http://gerrit.cloudera.org:8080/4477
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I497c1265df132fc8ea4e635475d0d669eca21646
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-HasComments: No

Reply via email to