[GitHub] [druid] mini666 commented on issue #9057: [new feature] Add HBase2 Indexer

[GitBox <gi...@apache.org>]

mini666 commented on issue #9057: [new feature] Add HBase2 Indexer
URL: https://github.com/apache/druid/pull/9057#issuecomment-584432373
 
 
   Hello, I agree with you.
   So I quickly looked at changing to InputSource.
   The bottom line is that changing to an InputSource is very influential. This is the amount that almost all new classes need to be created. Of course, the actual code won't change, but it must be moved to a new method in the new class. This will also lead to testing and documentation. I think it will take a long time to finish all this work. I don't expect exactly when it will end. 
   For now, in my opinion, it would be nice to include it in the main stream first and then move on.

----------------------------------------------------------------
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