Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[HOTFIX] Improve log message in CarbonWriterBuilder #2920

Closed
wants to merge 4 commits into from
Closed
Changes from 3 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Expand Up @@ -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 "
jackylk marked this conversation as resolved.
Show resolved Hide resolved
+ "set it, it can be the name of the application which is using the SDK");
}
CarbonLoadModel loadModel = buildLoadModel(schema);
loadModel.setSdkWriterCores(numOfThreads);
Expand Down