Add comment for clickhouse.map-string-as-varchar in ClickHouse tests

trinodb · Jan 19, 2022 · 2d6c0ae · 2d6c0ae
1 parent 6565276
commit 2d6c0ae
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/...clickhouse/src/test/java/io/trino/plugin/clickhouse/TestClickHouseConnectorSmokeTest.java b/...clickhouse/src/test/java/io/trino/plugin/clickhouse/TestClickHouseConnectorSmokeTest.java
@@ -34,7 +34,7 @@ protected QueryRunner createQueryRunner()
                 clickHouseServer,
                 ImmutableMap.of(),
                 ImmutableMap.<String, String>builder()
-                        .put("clickhouse.map-string-as-varchar", "true")
+                        .put("clickhouse.map-string-as-varchar", "true") // To handle string types in TPCH tables as varchar instead of varbinary
                         .buildOrThrow(),
                 REQUIRED_TPCH_TABLES);
     }

diff --git a/...ouse/src/test/java/io/trino/plugin/clickhouse/TestClickHouseLatestConnectorSmokeTest.java b/...ouse/src/test/java/io/trino/plugin/clickhouse/TestClickHouseLatestConnectorSmokeTest.java
@@ -30,7 +30,7 @@ protected QueryRunner createQueryRunner()
                 closeAfterClass(new TestingClickHouseServer(CLICKHOUSE_LATEST_IMAGE)),
                 ImmutableMap.of(),
                 ImmutableMap.<String, String>builder()
-                        .put("clickhouse.map-string-as-varchar", "true")
+                        .put("clickhouse.map-string-as-varchar", "true") // To handle string types in TPCH tables as varchar instead of varbinary
                         .buildOrThrow(),
                 REQUIRED_TPCH_TABLES);
     }