TAJO-1480: Kafka Consumer for kafka strage.#482
TAJO-1480: Kafka Consumer for kafka strage.#482hys9958 wants to merge 4 commits intoapache:masterfrom
Conversation
There was a problem hiding this comment.
It looks like a wrong version.
|
Oops! My mistake. |
There was a problem hiding this comment.
If users already ran above port for running zookeeper, this might be failed to run. Currently, Tajo provides hbase storage manager and it already implemented unit test cases with zookeeper. HBaseTestClusterUtil will be helpful for you to use zookeeper.
|
I found many dependencies as follows. I would like it more simple if you exclude unnecessary dependencies. |
|
@blrunner, |
There was a problem hiding this comment.
Tajo follows Sun's code convention style. You need update class description as follows.
/**
* SimpleConsumerManager is kafka...
*/
Please refer this site.
https://cwiki.apache.org/confluence/display/TAJO/Code+Style
Hi guys,
I have implemented for this issue and unit tests.
Please review my patch.
Thanks.