[
https://issues.apache.org/jira/browse/CALCITE-3540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16989778#comment-16989778
]
Stamatis Zampetakis commented on CALCITE-3540:
----------------------------------------------
Every method of AssertThat creates new object so I would say that it is normal
that pooled() creates a new pool every time it is called. I think that we
should change the way we use the AssertThat objects in FoodmartTest. I will
submit a PR soon.
> FoodmartTest produces many warnings due to incorrect use of
> CalciteAssert.pooled()
> ----------------------------------------------------------------------------------
>
> Key: CALCITE-3540
> URL: https://issues.apache.org/jira/browse/CALCITE-3540
> Project: Calcite
> Issue Type: Test
> Components: core
> Affects Versions: 1.21.0
> Reporter: Vladimir Sitnikov
> Assignee: Stamatis Zampetakis
> Priority: Major
> Labels: beginner
>
> Slow tests produce an extreme amount of warnings:
> Sample: [https://github.com/apache/calcite/runs/322567836#step:4:3681]
>
> org.apache.calcite.test.FoodmartTest > test(FoodmartQuery)[1116] STANDARD_OUT
> 2019-11-27 07:35:26,781 [pool-1116-thread-1] WARN -
> PoolableConnectionFactory not linked to pool. Calling setPool() to fix the
> configuration.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
