Use /tmp directory to transfer data from host to container

ClickHouse · Jun 26, 2022 · e86d45e · e86d45e
1 parent cfbfe13
commit e86d45e
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 10 deletions.
diff --git a/clickhouse-cli-client/src/main/java/com/clickhouse/client/cli/ClickHouseCommandLine.java b/clickhouse-cli-client/src/main/java/com/clickhouse/client/cli/ClickHouseCommandLine.java
@@ -144,7 +144,7 @@ static Process startProcess(ClickHouseNode server, ClickHouseRequest<?> request)
                 ClickHouseChecker.isNullOrBlank(hostDir) ? System.getProperty("java.io.tmpdir") : hostDir);
         String containerDir = (String) config.getOption(ClickHouseCommandLineOption.CLI_CONTAINER_DIRECTORY);
         if (ClickHouseChecker.isNullOrBlank(containerDir)) {
-            containerDir = "/data/";
+            containerDir = "/tmp/";
         } else {
             containerDir = ClickHouseUtils.normalizeDirectory(containerDir);
         }

diff --git a/...li-client/src/main/java/com/clickhouse/client/cli/config/ClickHouseCommandLineOption.java b/...li-client/src/main/java/com/clickhouse/client/cli/config/ClickHouseCommandLineOption.java
@@ -37,10 +37,10 @@ public enum ClickHouseCommandLineOption implements ClickHouseOption {
     /**
      * Work directory inside container, only works running in docker mode(when
      * {@link #CLICKHOUSE_CLI_PATH} is not available). Empty value is treated as
-     * '/data'.
+     * '/tmp'.
      */
     CLI_CONTAINER_DIRECTORY("cli_container_directory", "",
-            "Work directory inside container, empty value is treated as '/data'"),
+            "Work directory inside container, empty value is treated as '/tmp'"),
     /**
      * Command-line work directory. Empty value is treated as system temporary
      * directory(e.g. {@code System.getProperty("java.io.tmpdir")}). When running in

diff --git a/...e-cli-client/src/test/java/com/clickhouse/client/cli/ClickHouseCommandLineClientTest.java b/...e-cli-client/src/test/java/com/clickhouse/client/cli/ClickHouseCommandLineClientTest.java
@@ -47,12 +47,6 @@ protected ClickHouseNode getServer() {
         return super.getServer();
     }
 
-    @Test(groups = { "integration" })
-    @Override
-    public void testCustomLoad() throws Exception {
-        throw new SkipException("Skip due to time out error");
-    }
-
     @Test(groups = { "integration" })
     @Override
     public void testLoadRawData() throws Exception {

diff --git a/clickhouse-client/src/test/java/com/clickhouse/client/ClientIntegrationTest.java b/clickhouse-client/src/test/java/com/clickhouse/client/ClientIntegrationTest.java
@@ -975,7 +975,6 @@ public void testCustomWriter() throws Exception {
 
     @Test(groups = { "integration" })
     public void testDumpAndLoadFile() throws Exception {
-        // super.testLoadRawData();
         ClickHouseNode server = getServer();
         ClickHouseClient.send(server, "drop table if exists test_dump_load_file",
                 "create table test_dump_load_file(a UInt64, b Nullable(String)) engine=MergeTree() order by tuple()")