Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2920#discussion_r233470872
--- Diff:
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");
--- End diff --
```suggestion
+ "set it, it can be the name of the application which using
the SDK");
```
---
