xiedeyantu commented on code in PR #4751:
URL: https://github.com/apache/calcite/pull/4751#discussion_r2702402503
##########
core/src/test/java/org/apache/calcite/test/JdbcAdapterTest.java:
##########
@@ -181,23 +183,25 @@ class JdbcAdapterTest {
}
@Test void testInPlan() {
- CalciteAssert.model(FoodmartSchema.FOODMART_MODEL)
- .query("select \"store_id\", \"store_name\" from \"store\"\n"
- + "where \"store_name\" in ('Store 1', 'Store 10', 'Store 11',
'Store 15', 'Store 16', 'Store 24', 'Store 3', 'Store 7')")
- .runs()
- .enable(CalciteAssert.DB == CalciteAssert.DatabaseInstance.HSQLDB)
- .planHasSql("SELECT \"store_id\", \"store_name\"\n"
- + "FROM \"foodmart\".\"store\"\n"
- + "WHERE \"store_name\" IN ('Store 1', 'Store 10', 'Store 11',"
- + " 'Store 15', 'Store 16', 'Store 24', 'Store 3', 'Store 7')")
- .returns("store_id=1; store_name=Store 1\n"
- + "store_id=3; store_name=Store 3\n"
- + "store_id=7; store_name=Store 7\n"
- + "store_id=10; store_name=Store 10\n"
- + "store_id=11; store_name=Store 11\n"
- + "store_id=15; store_name=Store 15\n"
- + "store_id=16; store_name=Store 16\n"
- + "store_id=24; store_name=Store 24\n");
+ try (TryThreadLocal.Memo ignore =
Prepare.THREAD_INSUBQUERY_THRESHOLD.push(20)) {
Review Comment:
I added extra thread parameter configurations to ensure that this case can
be executed stably.
I think this PR is ready, although I can't guarantee whether other
non-Quidem tests will still have an impact. That might be something we need to
work on next.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
