From b7fd6a6a9469feb99fdbc6b82907bc5c57381792 Mon Sep 17 00:00:00 2001 From: Jacky Li Date: Wed, 14 Nov 2018 20:54:33 +0800 Subject: [PATCH 1/4] Update CarbonWriterBuilder.java --- .../org/apache/carbondata/sdk/file/CarbonWriterBuilder.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java b/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java index d3aaf3bc7bc..5b6fbdbd4d1 100644 --- a/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java +++ b/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java @@ -438,13 +438,13 @@ public CarbonWriter build() throws IOException, InvalidLoadOptionException { Objects.requireNonNull(path, "path should not be null"); if (this.writerType == null) { throw new IOException( - "Writer type is not set, use withCsvInput() or withAvroInput() or withJsonInput() " + "'writerType' must be set, use withCsvInput() or withAvroInput() or withJsonInput() " + "API based on input"); } if (this.writtenByApp == null || this.writtenByApp.isEmpty()) { throw new RuntimeException( - "AppName is not set, please use writtenBy() API to set the App Name" - + "which is using SDK"); + "'writtenBy' must be set when writting carbon files, use writtenBy() API to " + + "set it, it can be the application name which using the SDK"); } CarbonLoadModel loadModel = buildLoadModel(schema); loadModel.setSdkWriterCores(numOfThreads); From 882f6752e6a0a4df2c76591231cb30d13d78a5d8 Mon Sep 17 00:00:00 2001 From: Jacky Li Date: Wed, 14 Nov 2018 21:07:19 +0800 Subject: [PATCH 2/4] Update store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java --- .../org/apache/carbondata/sdk/file/CarbonWriterBuilder.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java b/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java index 5b6fbdbd4d1..e1ce018535e 100644 --- a/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java +++ b/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java @@ -443,7 +443,7 @@ public CarbonWriter build() throws IOException, InvalidLoadOptionException { } if (this.writtenByApp == null || this.writtenByApp.isEmpty()) { throw new RuntimeException( - "'writtenBy' must be set when writting carbon files, use writtenBy() API to " + "'writtenBy' must be set when writting carbon files, use writtenBy() API to " + "set it, it can be the application name which using the SDK"); } CarbonLoadModel loadModel = buildLoadModel(schema); From 7f6b6e9e27dd5e2a386a9e014d226c590813fe4a Mon Sep 17 00:00:00 2001 From: Jacky Li Date: Wed, 14 Nov 2018 22:34:49 +0800 Subject: [PATCH 3/4] Update store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java --- .../org/apache/carbondata/sdk/file/CarbonWriterBuilder.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java b/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java index e1ce018535e..8b50c568a39 100644 --- a/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java +++ b/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java @@ -444,7 +444,7 @@ public CarbonWriter build() throws IOException, InvalidLoadOptionException { if (this.writtenByApp == null || this.writtenByApp.isEmpty()) { throw new RuntimeException( "'writtenBy' must be set when writting carbon files, use writtenBy() API to " - + "set it, it can be the application name which using the SDK"); + + "set it, it can be the name of the application which is using the SDK"); } CarbonLoadModel loadModel = buildLoadModel(schema); loadModel.setSdkWriterCores(numOfThreads); From d12c0c9bdc77610edca4ff07bb2224088a5f0de1 Mon Sep 17 00:00:00 2001 From: xuchuanyin Date: Fri, 16 Nov 2018 14:54:22 +0800 Subject: [PATCH 4/4] Update store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java Co-Authored-By: jackylk --- .../org/apache/carbondata/sdk/file/CarbonWriterBuilder.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java b/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java index 8b50c568a39..2a77a84c802 100644 --- a/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java +++ b/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java @@ -443,7 +443,7 @@ public CarbonWriter build() throws IOException, InvalidLoadOptionException { } if (this.writtenByApp == null || this.writtenByApp.isEmpty()) { throw new RuntimeException( - "'writtenBy' must be set when writting carbon files, use writtenBy() API to " + "'writtenBy' must be set when writing carbon files, use writtenBy() API to " + "set it, it can be the name of the application which is using the SDK"); } CarbonLoadModel loadModel = buildLoadModel(schema);