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

rubenql pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/calcite.git


The following commit(s) were added to refs/heads/main by this push:
     new 267d633165 [CALCITE-5965] Avoid unnecessary String concatenations in 
the RexFieldAccess constructor to improve the performance
267d633165 is described below

commit 267d63316511289ebe91079714e43b4e2d2ba085
Author: Thomas Rebele <thomas.reb...@gmail.com>
AuthorDate: Tue Aug 29 15:21:37 2023 +0200

    [CALCITE-5965] Avoid unnecessary String concatenations in the 
RexFieldAccess constructor to improve the performance
---
 core/src/main/java/org/apache/calcite/rex/RexFieldAccess.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/main/java/org/apache/calcite/rex/RexFieldAccess.java 
b/core/src/main/java/org/apache/calcite/rex/RexFieldAccess.java
index 886f98f0e5..0f09672f38 100644
--- a/core/src/main/java/org/apache/calcite/rex/RexFieldAccess.java
+++ b/core/src/main/java/org/apache/calcite/rex/RexFieldAccess.java
@@ -75,7 +75,7 @@ public class RexFieldAccess extends RexNode {
     Preconditions.checkArgument(
         fieldIdx >= 0 && fieldIdx < exprType.getFieldList().size()
             && exprType.getFieldList().get(fieldIdx).equals(field),
-        "Field " + field + " does not exist for expression " + expr);
+        "Field %s does not exist for expression %s", field, expr);
   }
 
   public RelDataTypeField getField() {

Reply via email to