From ee8ddf1b60db6f5e90ae2fe5edb3276a45704f37 Mon Sep 17 00:00:00 2001
From: Martin Lowe <martin.lowe@eclipse-foundation.org>
Date: Tue, 19 Sep 2023 12:01:16 -0400
Subject: [PATCH] Add patch to fix limit use in loading GH install list
---
.../eca/service/impl/DefaultGithubApplicationService.java | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/src/main/java/org/eclipsefoundation/git/eca/service/impl/DefaultGithubApplicationService.java b/src/main/java/org/eclipsefoundation/git/eca/service/impl/DefaultGithubApplicationService.java
index ca5cd9e7..30e94ec8 100644
--- a/src/main/java/org/eclipsefoundation/git/eca/service/impl/DefaultGithubApplicationService.java
+++ b/src/main/java/org/eclipsefoundation/git/eca/service/impl/DefaultGithubApplicationService.java
@@ -148,7 +148,11 @@ public class DefaultGithubApplicationService implements GithubApplicationService
// build query to do fetch of records for currently active application
MultivaluedMap<String, String> params = new MultivaluedMapImpl<>();
params.add(GitEcaParameterNames.APPLICATION_ID_RAW, Integer.toString(config.github().appId()));
- return dao.get(new RDBMSQuery<>(wrap, filters.get(GithubApplicationInstallation.class), params));
+
+ // we don't want to use the limit, as we want to pull all of the records we can find
+ RDBMSQuery<GithubApplicationInstallation> q = new RDBMSQuery<>(wrap, filters.get(GithubApplicationInstallation.class), params);
+ q.setUseLimit(false);
+ return dao.get(q);
}
}
--
GitLab