You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2019/12/16 17:50:52 UTC

[GitHub] [incubator-druid] jihoonson commented on issue #9045: [new feature] Add HBase1 & HBase2 Indexer

jihoonson commented on issue #9045: [new feature] Add HBase1 & HBase2 Indexer
URL: https://github.com/apache/incubator-druid/pull/9045#issuecomment-566169049
 
 
   Hi @mini666, thanks for your contribution! I believe this extension would be super useful. Would you please considering splitting this PR into 2 smaller ones like one for adding support for HBase1 and another for HBase2? This PR is currently pretty big which could make review difficult.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org