[jira] [Commented] (STORM-2023) Add calcite-core to dependency of storm-sql-runtime

["Jungtaek Lim (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/STORM-2023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15409210#comment-15409210 ] 

Jungtaek Lim commented on STORM-2023:
-------------------------------------

It also needs to copy dependencies (sure transitive too) to dist vs assembly (and shade).
Since there's no clear decision made for shading non-storm-core modules, I'll follow storm-sql-core (copy dependencies) and revisit when we're clear on that.


> Add calcite-core to dependency of storm-sql-runtime
> ---------------------------------------------------
>
>                 Key: STORM-2023
>                 URL: https://issues.apache.org/jira/browse/STORM-2023
>             Project: Apache Storm
>          Issue Type: Bug
>          Components: storm-sql
>    Affects Versions: 1.0.0
>            Reporter: Jungtaek Lim
>            Assignee: Jungtaek Lim
>
> 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.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)