This is an automated email from the ASF dual-hosted git repository. snlee pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/pinot.git
commit 8961e5460f017c852bf3716a1e9fe411810abe1d Author: Aishik <[email protected]> AuthorDate: Mon Nov 6 19:35:15 2023 +0530 added check for illegal character '/' in taskName --- .../src/main/java/org/apache/pinot/spi/config/task/AdhocTaskConfig.java | 1 + 1 file changed, 1 insertion(+) diff --git a/pinot-spi/src/main/java/org/apache/pinot/spi/config/task/AdhocTaskConfig.java b/pinot-spi/src/main/java/org/apache/pinot/spi/config/task/AdhocTaskConfig.java index 3a1f5b1068..ad4af27646 100644 --- a/pinot-spi/src/main/java/org/apache/pinot/spi/config/task/AdhocTaskConfig.java +++ b/pinot-spi/src/main/java/org/apache/pinot/spi/config/task/AdhocTaskConfig.java @@ -57,6 +57,7 @@ public class AdhocTaskConfig extends BaseJsonConfig { @JsonProperty("taskConfigs") @Nullable Map<String, String> taskConfigs) { Preconditions.checkArgument(taskType != null, "'taskType' must be configured"); Preconditions.checkArgument(tableName != null, "'tableName' must be configured"); + Preconditions.checkArgument(taskName == null || !taskName.contains("/"), "'taskName' must not contain '/'"); _taskType = taskType; _tableName = tableName; _taskName = taskName; --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
