Github user rmetzger commented on the pull request:

    https://github.com/apache/flink/pull/1342#issuecomment-155768292
  
    Thanks a lot for working on this issue! It doesn't happen everyday that 
users identify issues, fix them and verify the fix so properly :)
    
    I would really like to avoid pulling hbase as dependency into Flink's 
binary distribution. Can you change to "obtain token" part to do everything by 
loading the classes dynamically and calling the methods via reflection? 
(Similar to Spark's implementation: 
https://github.com/apache/spark/pull/5586/files)
    This way, we only execute this if the hbase classes are in the classpath.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to