Skip to content

Conversation

@jgiovaresco
Copy link
Contributor

@jgiovaresco jgiovaresco commented Oct 28, 2025

Issue

https://gravitee.atlassian.net/browse/APIM-11726

Description

JDBC repository that fetches the latest events, where we are building an Event without all its properties. Therefore, when the API was synchronized on the Gateway, the revision was not defined, and when an update was deployed, it was ignored because the revision number was the same.

The fix was not straightforward and may not be efficient. If we notice performance issues during the sync, we may need to investigate more on how we can improve this JdbcEventLatestRepository

Additional context

@cypress
Copy link

cypress bot commented Oct 28, 2025

APIM UI Tests    Run #6670

Run Properties:  status check passed Passed #6670  •  git commit 98a589ddf2: fix: jdbc repository return latest event with all their properties
Project APIM UI Tests
Branch Review apim11726-run-e2e
Run status status check passed Passed #6670
Run duration 07m 02s
Commit git commit 98a589ddf2: fix: jdbc repository return latest event with all their properties
Committer Julien Giovaresco
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 8
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 82
View all changes introduced in this branch ↗︎

@jgiovaresco jgiovaresco marked this pull request as ready for review October 28, 2025 21:54
@jgiovaresco jgiovaresco requested a review from a team as a code owner October 28, 2025 21:54
@jgiovaresco jgiovaresco merged commit 18dad2a into 4.9.x Oct 29, 2025
55 of 56 checks passed
@jgiovaresco jgiovaresco deleted the apim11726-run-e2e branch October 29, 2025 09:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants