From 72ab993dc8f448dcdb84716bdef2e0f22d209d07 Mon Sep 17 00:00:00 2001 From: Parampreet Singh Chahal <164636326+parampreetchahal@users.noreply.github.com> Date: Sat, 8 Feb 2025 23:40:43 +0530 Subject: [PATCH] Update Capability.java --- .../connector/capability/Capability.java | 29 +++++++++++-------- 1 file changed, 17 insertions(+), 12 deletions(-) diff --git a/core/src/main/java/org/apache/gravitino/connector/capability/Capability.java b/core/src/main/java/org/apache/gravitino/connector/capability/Capability.java index 37d517f2d21..9d89902ef29 100644 --- a/core/src/main/java/org/apache/gravitino/connector/capability/Capability.java +++ b/core/src/main/java/org/apache/gravitino/connector/capability/Capability.java @@ -126,18 +126,23 @@ public CapabilityResult caseSensitiveOnName(Scope scope) { } @Override - public CapabilityResult specificationOnName(Scope scope, String name) { - if (RESERVED_WORDS.contains(name.toLowerCase())) { - return CapabilityResult.unsupported( - String.format("The %s name '%s' is reserved.", scope, name)); - } - - if (!name.matches(DEFAULT_NAME_PATTERN)) { - return CapabilityResult.unsupported( - String.format("The %s name '%s' is illegal.", scope, name)); - } - return CapabilityResult.SUPPORTED; - } +public CapabilityResult specificationOnName(Scope scope, String name) { + if (RESERVED_WORDS.contains(name.toLowerCase())) { + return CapabilityResult.unsupported( + String.format("The %s name '%s' is reserved.", scope, name)); + } + + if (!name.matches(DEFAULT_NAME_PATTERN)) { + return CapabilityResult.unsupported( + String.format( + "The %s name '%s' is illegal. It must start with a letter, digit, or underscore, " + + "and may contain letters, digits, underscores, slashes, hyphens, or equals signs, " + + "with a maximum length of 64 characters.", + scope, name)); + } + return CapabilityResult.SUPPORTED; +} + @Override public CapabilityResult managedStorage(Scope scope) {