Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2652#discussion_r213946688
--- Diff:
store/sdk/src/main/java/org/apache/carbondata/sdk/file/CarbonWriterBuilder.java
---
@@ -246,8 +246,8 @@ public CarbonWriterBuilder withLoadOptions(Map<String,
String> options) {
Objects.requireNonNull(options, "Load options should not be null");
//validate the options.
if (options.size() > 9) {
- throw new IllegalArgumentException("Supports only nine options now. "
- + "Refer method header or documentation");
+ throw new IllegalArgumentException(
+ "Supports only ten options now. Refer method header or
documentation");
--- End diff --
I think validating in this way is not good. @KanakaKumar can you have a
check?
---
