(pinot) 01/02: added check for illegal character '/' in taskName

[sn...@apache.org]

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 <ai...@startree.ai>
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: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org