[STORM-2023] Add calcite-core to dependency of storm-sql-runtime

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.0.0
• Fix Version/s: 2.0.0, 1.1.0
• Component/s: storm-sql
• Epic Link: SQL support for Storm Phase II -- support aggregations and simplify deployment
• Assignee: Jungtaek Lim
• Reporter: Jungtaek Lim
• Created: 05/Aug/16 06:24
• Updated: 12/Aug/16 04:09
• Resolved: 12/Aug/16 04:09

Description

storm-sql provides both "storm-sql-core" and "storm-sql-runtime". Former is for compiling sql to trident topology, and latter is for running compiled trident topology.

While testing storm-sql feature I found compiled class codes refers calcite so calcite-core is needed at runtime.
I'm not sure we can make calcite-core get out of storm-sql-runtime, so for now we can add calcite-core to storm-sql-runtime so that it can be available on transitive dependencies for storm-sql-runtime.

Issue Links

• is related to: CALCITE-1349 Make runtime packages less depend on other libraries
• links to: GitHub Pull Request #1610