[jackrabbit-oak] branch OAK-9780 updated: OAK-9780 PoC for DocumentNodeStore cache warming - avoid unnecessary conversion

[th...@apache.org]

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

thomasm pushed a commit to branch OAK-9780
in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git


The following commit(s) were added to refs/heads/OAK-9780 by this push:
     new 89f8fe9cad OAK-9780 PoC for DocumentNodeStore cache warming - avoid unnecessary conversion
89f8fe9cad is described below

commit 89f8fe9cad509175ae52d1d8a1172536c61131c1
Author: Thomas Mueller <th...@apache.org>
AuthorDate: Fri Jun 24 10:35:10 2022 +0200

    OAK-9780 PoC for DocumentNodeStore cache warming - avoid unnecessary conversion
---
 .../src/main/java/org/apache/jackrabbit/oak/query/QueryOptions.java  | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryOptions.java b/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryOptions.java
index 832fa6683c..d244378f46 100644
--- a/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryOptions.java
+++ b/oak-core/src/main/java/org/apache/jackrabbit/oak/query/QueryOptions.java
@@ -117,6 +117,11 @@ public class QueryOptions {
         }
 
         public QueryOptions getDefaultValues(String statement) {
+            if (map.isEmpty()) {
+                // no need to simplify if there are no entries
+                // (which is probably most of the time)
+                return new QueryOptions();
+            }
             String queryPattern = QueryRecorder.simplifySafely(statement);
             return map.computeIfAbsent(queryPattern, v -> new QueryOptions());
         }