[
https://issues.apache.org/jira/browse/STORM-944?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14681619#comment-14681619
]
ASF GitHub Bot commented on STORM-944:
--------------------------------------
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.
> storm-hive pom.xml has a dependency conflict with calcite
> ---------------------------------------------------------
>
> Key: STORM-944
> URL: https://issues.apache.org/jira/browse/STORM-944
> Project: Apache Storm
> Issue Type: Bug
> Components: external
> Reporter: Aaron Dossett
> Assignee: Aaron Dossett
> Priority: Trivial
>
> Hive 0.14.0 has a dependency on calcite-0.9.2-incubating-SNAPSHOT which can't
> be resolved in maven central. See HIVE-8906 for details.
> This gives a harmless compile warning for storm-hive but it does prevent some
> IDEs (IntelliJ for certain, probably others) from correctly resolving the
> project dependencies. storm-hive already has a dependency on
> calcite-0.9.2-incubating so calcite should be excluded from the hive
> dependency.
> Compile warning:
> [WARNING] Missing POM for
> org.apache.calcite:calcite-core:jar:0.9.2-incubating-SNAPSHOT
> [WARNING] Missing POM for
> org.apache.calcite:calcite-avatica:jar:0.9.2-incubating-SNAPSHOT
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
