From 00bfc2c34f7aa173400dea6ccbd61445059e7026 Mon Sep 17 00:00:00 2001
From: Alexander Schwartz <aschwart@redhat.com>
Date: Thu, 1 Aug 2024 11:12:53 +0200
Subject: [PATCH] Adding an index for the revoked tokens table to speed up the
 cleanup (#31790)

Closes #31725

Signed-off-by: Alexander Schwartz <aschwart@redhat.com>
---
 .../InfinispanSingleUseObjectProviderFactory.java           | 2 +-
 .../src/main/resources/META-INF/jpa-changelog-26.0.0.xml    | 6 ++++++
 2 files changed, 7 insertions(+), 1 deletion(-)

diff --git a/model/infinispan/src/main/java/org/keycloak/models/sessions/infinispan/InfinispanSingleUseObjectProviderFactory.java b/model/infinispan/src/main/java/org/keycloak/models/sessions/infinispan/InfinispanSingleUseObjectProviderFactory.java
index 839b817d81..e5944c322d 100644
--- a/model/infinispan/src/main/java/org/keycloak/models/sessions/infinispan/InfinispanSingleUseObjectProviderFactory.java
+++ b/model/infinispan/src/main/java/org/keycloak/models/sessions/infinispan/InfinispanSingleUseObjectProviderFactory.java
@@ -117,7 +117,7 @@ public class InfinispanSingleUseObjectProviderFactory implements SingleUseObject
     @Override
     public void postInit(KeycloakSessionFactory factory) {
         // It is necessary to put the cache initialization here, otherwise the cache would be initialized lazily, that
-        // means also listeners will start only after first cache initialization - that would be too latedddd
+        // means also listeners will start only after first cache initialization - that would be too late
         if (singleUseObjectCache == null) {
             this.singleUseObjectCache = getSingleUseObjectCache(factory.create());
         }
diff --git a/model/jpa/src/main/resources/META-INF/jpa-changelog-26.0.0.xml b/model/jpa/src/main/resources/META-INF/jpa-changelog-26.0.0.xml
index 50de3d6270..ccf78d1e2b 100644
--- a/model/jpa/src/main/resources/META-INF/jpa-changelog-26.0.0.xml
+++ b/model/jpa/src/main/resources/META-INF/jpa-changelog-26.0.0.xml
@@ -67,4 +67,10 @@
         <addPrimaryKey columnNames="ID" constraintName="CONSTRAINT_RT" tableName="REVOKED_TOKEN"/>
     </changeSet>
 
+    <changeSet author="keycloak" id="31725-index-persist-revoked-access-tokens">
+        <createIndex indexName="IDX_REV_TOKEN_ON_EXPIRE" tableName="REVOKED_TOKEN">
+            <column name="EXPIRE" />
+        </createIndex>
+    </changeSet>
+
 </databaseChangeLog>