Code review comment: Need to trim the space, eg: `"delete, sequence" => "delete,sequence"`

Code before review:
private void setOptionalFromTSLPutRequest(TStreamLoadPutRequest request) throws UserException {
if (request.isSetColumns()) {
setColumnToColumnExpr(request.getColumns());
}
if (request.isSetWhere()) {
whereExpr = parseWhereExpr(request.getWhere());
}
if (request.isSetColumnSeparator()) {
setColumnSeparator(request.getColumnSeparator());
}
if (request.isSetLineDelimiter()) {
setLineDelimiter(request.getLineDelimiter());
}
if (request.isSetHeaderType()) {
headerType = request.getHeaderType();
}
if (request.isSetPartitions()) {
String[] partNames = request.getPartitions().trim().split("\\s*,\\s*");
if (request.isSetIsTempPartition()) {
partitions = new PartitionNames(request.isIsTempPartition(), Lists.newArrayList(partNames));
} else {
partitions = new PartitionNames(false, Lists.newArrayList(partNames));
}
}
switch (request.getFileType()) {
case FILE_STREAM:
case FILE_LOCAL:
path = request.getPath();
break;
default:
throw new UserException("unsupported file type, type=" + request.getFileType());
}
if (request.isSetNegative()) {
negative = request.isNegative();
}
if (request.isSetTimeout()) {
timeout = request.getTimeout();
}
if (request.isSetStrictMode()) {
strictMode = request.isStrictMode();
}
if (request.isSetTimezone()) {
timezone = TimeUtils.checkTimeZoneValidAndStandardize(request.getTimezone());
}
if (request.isSetExecMemLimit()) {
execMemLimit = request.getExecMemLimit();
}
if (request.getFormatType() == TFileFormatType.FORMAT_JSON) {
if (request.getJsonpaths() != null) {
jsonPaths = request.getJsonpaths();
}
if (request.getJsonRoot() != null) {
jsonRoot = request.getJsonRoot();
}
stripOuterArray = request.isStripOuterArray();
numAsString = request.isNumAsString();
fuzzyParse = request.isFuzzyParse();
readJsonByLine = request.isReadJsonByLine();
}
if (request.isSetMergeType()) {
try {
mergeType = LoadTask.MergeType.valueOf(request.getMergeType().toString());
} catch (IllegalArgumentException e) {
throw new UserException("unknown merge type " + request.getMergeType().toString());
}
}
if (request.isSetDeleteCondition()) {
deleteCondition = parseWhereExpr(request.getDeleteCondition());
}
if (negative && mergeType != LoadTask.MergeType.APPEND) {
throw new AnalysisException("Negative is only used when merge type is APPEND.");
}
if (request.isSetSequenceCol()) {
sequenceCol = request.getSequenceCol();
}
if (request.isSetSendBatchParallelism()) {
sendBatchParallelism = request.getSendBatchParallelism();
}
if (request.isSetMaxFilterRatio()) {
maxFilterRatio = request.getMaxFilterRatio();
}
if (request.isSetLoadToSingleTablet()) {
loadToSingleTablet = request.isLoadToSingleTablet();
}
if (request.isSetHiddenColumns()) {
hiddenColumns = Arrays.asList(request.getHiddenColumns().split(","));
}
}

Line under review: hiddenColumns = Arrays.asList(request.getHiddenColumns().split(","));

Code after addressing the review:
private void setOptionalFromTSLPutRequest(TStreamLoadPutRequest request) throws UserException {
if (request.isSetColumns()) {
setColumnToColumnExpr(request.getColumns());
}
if (request.isSetWhere()) {
whereExpr = parseWhereExpr(request.getWhere());
}
if (request.isSetColumnSeparator()) {
setColumnSeparator(request.getColumnSeparator());
}
if (request.isSetLineDelimiter()) {
setLineDelimiter(request.getLineDelimiter());
}
if (request.isSetHeaderType()) {
headerType = request.getHeaderType();
}
if (request.isSetPartitions()) {
String[] partNames = request.getPartitions().trim().split("\\s*,\\s*");
if (request.isSetIsTempPartition()) {
partitions = new PartitionNames(request.isIsTempPartition(), Lists.newArrayList(partNames));
} else {
partitions = new PartitionNames(false, Lists.newArrayList(partNames));
}
}
switch (request.getFileType()) {
case FILE_STREAM:
case FILE_LOCAL:
path = request.getPath();
break;
default:
throw new UserException("unsupported file type, type=" + request.getFileType());
}
if (request.isSetNegative()) {
negative = request.isNegative();
}
if (request.isSetTimeout()) {
timeout = request.getTimeout();
}
if (request.isSetStrictMode()) {
strictMode = request.isStrictMode();
}
if (request.isSetTimezone()) {
timezone = TimeUtils.checkTimeZoneValidAndStandardize(request.getTimezone());
}
if (request.isSetExecMemLimit()) {
execMemLimit = request.getExecMemLimit();
}
if (request.getFormatType() == TFileFormatType.FORMAT_JSON) {
if (request.getJsonpaths() != null) {
jsonPaths = request.getJsonpaths();
}
if (request.getJsonRoot() != null) {
jsonRoot = request.getJsonRoot();
}
stripOuterArray = request.isStripOuterArray();
numAsString = request.isNumAsString();
fuzzyParse = request.isFuzzyParse();
readJsonByLine = request.isReadJsonByLine();
}
if (request.isSetMergeType()) {
try {
mergeType = LoadTask.MergeType.valueOf(request.getMergeType().toString());
} catch (IllegalArgumentException e) {
throw new UserException("unknown merge type " + request.getMergeType().toString());
}
}
if (request.isSetDeleteCondition()) {
deleteCondition = parseWhereExpr(request.getDeleteCondition());
}
if (negative && mergeType != LoadTask.MergeType.APPEND) {
throw new AnalysisException("Negative is only used when merge type is APPEND.");
}
if (request.isSetSequenceCol()) {
sequenceCol = request.getSequenceCol();
}
if (request.isSetSendBatchParallelism()) {
sendBatchParallelism = request.getSendBatchParallelism();
}
if (request.isSetMaxFilterRatio()) {
maxFilterRatio = request.getMaxFilterRatio();
}
if (request.isSetLoadToSingleTablet()) {
loadToSingleTablet = request.isLoadToSingleTablet();
}
if (request.isSetHiddenColumns()) {
hiddenColumns = Arrays.asList(request.getHiddenColumns().replaceAll("\\s+", "").split(","));
}
}