GH-1556 Fix aggregation query in `SqlStatisticsRepository.findResolve…

…dRequestsByPhrase` used by `stats` command (Fix #1556)
dzikoysk · Oct 5, 2022 · 8c5be6f · 8c5be6f
1 parent 0334a16
commit 8c5be6f
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/...ckend/src/main/kotlin/com/reposilite/statistics/infrastructure/SqlStatisticsRepository.kt b/...ckend/src/main/kotlin/com/reposilite/statistics/infrastructure/SqlStatisticsRepository.kt
@@ -119,7 +119,7 @@ internal class SqlStatisticsRepository(private val database: Database) : Statist
             IdentifierTable.leftJoin(ResolvedTable, { IdentifierTable.id }, { ResolvedTable.identifierId })
                 .slice(IdentifierTable.gav, resolvedSum)
                 .select(whereCriteria)
-                .groupBy(ResolvedTable.id, IdentifierTable.gav)
+                .groupBy(IdentifierTable.id, IdentifierTable.gav)
                 .having { resolvedSum greater 0L }
                 .orderBy(resolvedSum, DESC)
                 .limit(limit)

diff --git a/reposilite-backend/src/test/kotlin/com/reposilite/maven/PreservedBuildsListenerTest.kt b/reposilite-backend/src/test/kotlin/com/reposilite/maven/PreservedBuildsListenerTest.kt
@@ -86,7 +86,7 @@ internal class PreservedBuildsListenerTest : MavenSpecification() {
                 "$versionId/maven-metadata.xml.md5",
                 "$versionId/maven-metadata.xml.sha1",
                 "$versionId/maven-metadata.xml.sha256",
-                "$versionId/maven-metadata.xml.sha512"
+                "$versionId/workspace-deploy-app-1.0.0.jar"
             ).map { it.toLocation() },
             assertOk(repository.getFiles(versionId))
         )

diff --git a/reposilite-backend/src/test/workspace/configuration.cdn b/reposilite-backend/src/test/workspace/configuration.cdn
@@ -52,7 +52,7 @@ idleTimeout: 30000
 
 # Keep processed frontend files in memory to improve response time
 cacheContent: true
-# Adds cache bypass headers to each request served by this instance.
+# Adds cache bypass headers to each request from /api/* scope served by this instance.
 # Helps to avoid various random issues caused by proxy provides (e.g. Cloudflare) and browsers.
 bypassExternalCache: true
 # Amount of messages stored in cached logger.