From 6c2c9b83edff20bf9588e45faf3830e388071d91 Mon Sep 17 00:00:00 2001 From: Darin Kelkhoff Date: Fri, 29 Sep 2023 17:08:24 -0500 Subject: [PATCH] Micro optimization in hot-spot - setValueIfTableHasField - use fields.containsKey, rather than getField, which throws, and is expensive when so frequent --- .../qqq/backend/module/rdbms/actions/AbstractRDBMSAction.java | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/qqq-backend-module-rdbms/src/main/java/com/kingsrook/qqq/backend/module/rdbms/actions/AbstractRDBMSAction.java b/qqq-backend-module-rdbms/src/main/java/com/kingsrook/qqq/backend/module/rdbms/actions/AbstractRDBMSAction.java index 6037ad75..75078c8a 100644 --- a/qqq-backend-module-rdbms/src/main/java/com/kingsrook/qqq/backend/module/rdbms/actions/AbstractRDBMSAction.java +++ b/qqq-backend-module-rdbms/src/main/java/com/kingsrook/qqq/backend/module/rdbms/actions/AbstractRDBMSAction.java @@ -193,8 +193,7 @@ public abstract class AbstractRDBMSAction implements QActionInterface { try { - QFieldMetaData field = table.getField(fieldName); - if(field != null) + if(table.getFields().containsKey(fieldName)) { record.setValue(fieldName, value); }