(ignite-3) branch main updated: IGNITE-25011 Make IgniteNameUtils.quoteIfNeeded consistent with parse methods (#5559)

[amashenkov]

This is an automated email from the ASF dual-hosted git repository.

amashenkov pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/ignite-3.git


The following commit(s) were added to refs/heads/main by this push:
     new e11fc77b282 IGNITE-25011 Make IgniteNameUtils.quoteIfNeeded consistent 
with parse methods (#5559)
e11fc77b282 is described below

commit e11fc77b282e44634c25dd0bbc33d36b25815b85
Author: Andrew V. Mashenkov <amashen...@users.noreply.github.com>
AuthorDate: Fri Apr 4 14:40:15 2025 +0300

    IGNITE-25011 Make IgniteNameUtils.quoteIfNeeded consistent with parse 
methods (#5559)
---
 .../api/src/main/java/org/apache/ignite/lang/util/IgniteNameUtils.java  | 2 +-
 .../test/java/org/apache/ignite/internal/util/IgniteNameUtilsTest.java  | 2 ++
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git 
a/modules/api/src/main/java/org/apache/ignite/lang/util/IgniteNameUtils.java 
b/modules/api/src/main/java/org/apache/ignite/lang/util/IgniteNameUtils.java
index 082cf731c1e..e77aa10bd1e 100644
--- a/modules/api/src/main/java/org/apache/ignite/lang/util/IgniteNameUtils.java
+++ b/modules/api/src/main/java/org/apache/ignite/lang/util/IgniteNameUtils.java
@@ -93,7 +93,7 @@ public final class IgniteNameUtils {
 
         int codePoint = identifier.codePointAt(0);
 
-        if (!(Character.isUpperCase(codePoint) && identifierStart(codePoint))) 
{
+        if (codePoint != '_' && !(Character.isUpperCase(codePoint) && 
identifierStart(codePoint))) {
             return quote(identifier);
         }
 
diff --git 
a/modules/core/src/test/java/org/apache/ignite/internal/util/IgniteNameUtilsTest.java
 
b/modules/core/src/test/java/org/apache/ignite/internal/util/IgniteNameUtilsTest.java
index d17b8bded6c..782a329a87b 100644
--- 
a/modules/core/src/test/java/org/apache/ignite/internal/util/IgniteNameUtilsTest.java
+++ 
b/modules/core/src/test/java/org/apache/ignite/internal/util/IgniteNameUtilsTest.java
@@ -113,6 +113,8 @@ public class IgniteNameUtilsTest {
                 Arguments.of("F16", "F16"),
                 Arguments.of("Ff16", "\"Ff16\""),
                 Arguments.of("FF16", "FF16"),
+                Arguments.of("_FF16", "_FF16"),
+                Arguments.of("FF_16", "FF_16"),
                 Arguments.of(" ", "\" \""),
                 Arguments.of(" F", "\" F\""),
                 Arguments.of(" ,", "\" ,\""),