>From Janhavi Tripurwar <[email protected]>:

Janhavi Tripurwar has submitted this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20484?usp=email )

Change subject: [ASTERIXDB-3664]: Simplify function name handling in 
EntityDetails
......................................................................

[ASTERIXDB-3664]: Simplify function name handling in EntityDetails

- user model changes: no
- storage format changes: no
- interface changes: no

Ext-ref: MB-68413

Change-Id: Ie3f02d8b3bb8484c413447cf2631b462ce20003a
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20484
Tested-by: Jenkins <[email protected]>
Reviewed-by: Ali Alsuliman <[email protected]>
Integration-Tests: Jenkins <[email protected]>
---
M 
asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entities/EntityDetails.java
1 file changed, 2 insertions(+), 4 deletions(-)

Approvals:
  Jenkins: Verified; Verified
  Ali Alsuliman: Looks good to me, approved




diff --git 
a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entities/EntityDetails.java
 
b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entities/EntityDetails.java
index 0b82c38..a61de11 100644
--- 
a/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entities/EntityDetails.java
+++ 
b/asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/entities/EntityDetails.java
@@ -92,13 +92,11 @@
         String functionName = fs.getName();
         Integer functionArity = fs.getArity();
         DataverseName dataverseName = fs.getDataverseName();
-        String functionNameWithArity = getFunctionNameWithArity(functionName, 
functionArity);
         if (isBuiltInFunc) {
-            return new EntityDetails(databaseName, dataverseName, 
functionNameWithArity, EntityType.BUILT_IN_FUNCTION,
+            return new EntityDetails(databaseName, dataverseName, 
functionName, EntityType.BUILT_IN_FUNCTION,
                     functionArity);
         }
-        return new EntityDetails(databaseName, dataverseName, 
functionNameWithArity, EntityType.FUNCTION,
-                functionArity);
+        return new EntityDetails(databaseName, dataverseName, functionName, 
EntityType.FUNCTION, functionArity);
     }

     public static EntityDetails newSynonym(String databaseName, DataverseName 
dataverseName, String synonymName) {

--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20484?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: asterixdb
Gerrit-Branch: phoenix
Gerrit-Change-Id: Ie3f02d8b3bb8484c413447cf2631b462ce20003a
Gerrit-Change-Number: 20484
Gerrit-PatchSet: 8
Gerrit-Owner: Janhavi Tripurwar <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Janhavi Tripurwar <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Reviewer: Murtadha Hubail <[email protected]>

Reply via email to