[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17515158#comment-17515158
]
Ivan Daschinsky commented on IGNITE-16632:
--
[~vveider] Hi, seems we have forgotten to add one more jar in order to calcite
module work properly in ODBC test suite. Could you please add this line to *>
Build* in 10-th step
{code}
zip -r ignite target examples/target modules/{*,*/*}/target -x '*.jar'
zip -ur ignite modules/{*,*/*}/target -i */aspectjweaver-*.jar \
-i */cache-api-*.jar \
-i */h2-*.jar \
-i */commons-codec-*.jar # this one
{code}
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17508882#comment-17508882
]
Aleksey Plekhanov commented on IGNITE-16632:
[~vveider], thanks!
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17508251#comment-17508251
]
Petr Ivanov commented on IGNITE-16632:
--
Updated.
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17508239#comment-17508239
]
Aleksey Plekhanov commented on IGNITE-16632:
[~vveider] can you please change the set of libraries to include to ignite.zip
file?
After changes to ODBC some additional libraries are still missing in classpath
(\{{jackson*}}, perhaps some other). We've changed script on TC2:
{noformat}
if [ -d "modules/calcite" ]; then
zip -ur ignite modules/calcite/target/libs/*.jar
fi{noformat}
And it works now. Can you please make such changes on TC1?
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17501012#comment-17501012
]
Aleksey Plekhanov commented on IGNITE-16632:
[~alex_pl] tests passed, thanks for the fix!
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17500579#comment-17500579
]
Petr Ivanov commented on IGNITE-16632:
--
Added rules to ignite.zip (run is used only for starting nodes from console).
[~alex_pl], check, please.
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17499427#comment-17499427
]
Aleksey Plekhanov commented on IGNITE-16632:
[~vveider], one more suite with the same problem: "Start Nodes"
([https://ci.ignite.apache.org/viewType.html?buildTypeId=IgniteTests24Java8_StartNodes])
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17498846#comment-17498846
]
Aleksey Plekhanov commented on IGNITE-16632:
[~vveider], currently, I've faced with this problem only for C++ builds
(Platform C++ CMake (Win x64 | Release), Platform C++ CMake (Linux), Platform
C++ CMake (Linux CLang))
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17498840#comment-17498840
]
Petr Ivanov commented on IGNITE-16632:
--
[~alex_pl] I ment which build configuration will use these new artifacts?
I do not want to increase in size current ignite.zip, so I want to create
separate archive and use it only in target test build configurations.
Which one will use them?
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17498839#comment-17498839
]
Aleksey Plekhanov commented on IGNITE-16632:
[~vveider], build config
([https://ci.ignite.apache.org/admin/editBuildRunners.html?id=buildType:IgniteTests24Java8_Build]),
step 10.
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17498837#comment-17498837
]
Petr Ivanov commented on IGNITE-16632:
--
[~alex_pl] which exactly config should be updated to use these artifacts?
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Assignee: Petr Ivanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
[jira] [Commented] (IGNITE-16632) Calcite engine. Add calcite dependencies to TC build
[
https://issues.apache.org/jira/browse/IGNITE-16632?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17498819#comment-17498819
]
Aleksey Plekhanov commented on IGNITE-16632:
[~vveider], can you please change the settings?
> Calcite engine. Add calcite dependencies to TC build
>
>
> Key: IGNITE-16632
> URL: https://issues.apache.org/jira/browse/IGNITE-16632
> Project: Ignite
> Issue Type: Task
>Reporter: Aleksey Plekhanov
>Priority: Major
> Labels: build, calcite
>
> Currently, a limited set of JARs is included in build artifacts
> ([https://ci.ignite.apache.org/admin/editRunType.html?id=buildType:IgniteTests24Java8_Build=RUNNER_286]).
> There are some new external dependencies for the new calcite module, that
> should be added too. Some test suites can't pass without these dependencies
> (C++ for example: [https://ci.ignite.apache.org/viewLog.html?buildId=6435330])
> {noformat}
> zip -ur run modules/calcite/target/libs/calcite-*.jar \
> modules/calcite/target/libs/avatica-*.jar \
> modules/calcite/target/libs/checker-qual-*.jar \
> modules/calcite/target/libs/commons-compiler-*.jar \
> modules/calcite/target/libs/esri-geometry-*.jar \
> modules/calcite/target/libs/guava-*.jar \
> modules/calcite/target/libs/janino-*.jar \
> modules/calcite/target/libs/json-path-*.jar {noformat}
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
