Github user HeartSaVioR commented on the pull request:
https://github.com/apache/storm/pull/609#issuecomment-129836616
@dossett
Makes sense.
Last thing I'd like to know is that current patch can work with Hive 1.x.y.
If it isn't, storm-hive is just for 0.14.0 since you showed storm-hive
doesn't work with Hive 0.13.0.
It would be better to specify supporting Hive versions to README.
@caofangkun
If it doesn't affect storm-hive, I also think it would be better to remove
explicit dependency about calcite, since higher version of Hive could require
higher version of calcite.
But as I stated earlier, I didn't use Hive so verification and confirmation
from Committer Sponsors are required.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
