[44/50] [abbrv] carbondata git commit: [HOTFIX] Improve log message in CarbonWriterBuilder

2018-11-21 Thread ravipesala 
[HOTFIX] Improve log message in CarbonWriterBuilder

In master the log message is not proper:
AppName is not set, please use writtenBy() API to set the App Namewhich is 
using SDK

This closes #2920


Project: http://git-wip-us.apache.org/repos/asf/carbondata/repo
Commit: http://git-wip-us.apache.org/repos/asf/carbondata/commit/d4bd4ab6
Tree: http://git-wip-us.apache.org/repos/asf/carbondata/tree/d4bd4ab6
Diff: http://git-wip-us.apache.org/repos/asf/carbondata/diff/d4bd4ab6

Branch: refs/heads/branch-1.5
Commit: d4bd4ab61521fce160f8980839a2ee2a9f2164c6
Parents: 2f7b003
Author: Jacky Li 
Authored: Wed Nov 14 20:54:33 2018 +0800
Committer: ravipesala 
Committed: Wed Nov 21 22:43:46 2018 +0530

--
 .../org/apache/carbondata/sdk/file/CarbonWriterBuilder.java| 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/carbondata/blob/d4bd4ab6/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
--
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 917d4dc..1ca5b74 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 class CarbonWriterBuilder {
 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 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);
 loadModel.setSdkWriterCores(numOfThreads);

carbondata git commit: [HOTFIX] Improve log message in CarbonWriterBuilder

2018-11-21 Thread xuchuanyin 
Repository: carbondata
Updated Branches:
  refs/heads/master c2ae98744 -> 3b8de320d


[HOTFIX] Improve log message in CarbonWriterBuilder

In master the log message is not proper:
AppName is not set, please use writtenBy() API to set the App Namewhich is 
using SDK

This closes #2920


Project: http://git-wip-us.apache.org/repos/asf/carbondata/repo
Commit: http://git-wip-us.apache.org/repos/asf/carbondata/commit/3b8de320
Tree: http://git-wip-us.apache.org/repos/asf/carbondata/tree/3b8de320
Diff: http://git-wip-us.apache.org/repos/asf/carbondata/diff/3b8de320

Branch: refs/heads/master
Commit: 3b8de320d7092470e6d58ad3dcee594e3ae7ecc8
Parents: c2ae987
Author: Jacky Li 
Authored: Wed Nov 14 20:54:33 2018 +0800
Committer: xuchuanyin 
Committed: Wed Nov 21 18:32:26 2018 +0800

--
 .../org/apache/carbondata/sdk/file/CarbonWriterBuilder.java| 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/carbondata/blob/3b8de320/store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
--
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 917d4dc..1ca5b74 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 class CarbonWriterBuilder {
 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 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);
 loadModel.setSdkWriterCores(numOfThreads);