Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Kv store query alignment #794

Merged
merged 3 commits into from
Oct 3, 2024
Merged

Kv store query alignment #794

merged 3 commits into from
Oct 3, 2024

Conversation

PawelPawelec-RDX
Copy link
Contributor

No description provided.

@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 18, 2024 •
edited
Loading

Docker tags
docker.io/radixdlt/private-babylon-ng-gateway-api:HEAD-5cc6de1-arm64

@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 18, 2024 •
edited
Loading

Docker tags
docker.io/radixdlt/private-babylon-ng-database-migrations:HEAD-5cc6de1-arm64

@github-actions GitHub Actions
Copy link

github-actions bot commented Sep 18, 2024 •
edited
Loading

Docker tags
docker.io/radixdlt/private-babylon-ng-data-aggregator:HEAD-5cc6de1-arm64

@PawelPawelec-RDX PawelPawelec-RDX force-pushed the kv-store-query-alignment branch 2 times, most recently from 943e195 to 293899a Compare September 19, 2024 09:44
@PawelPawelec-RDX PawelPawelec-RDX changed the base branch from develop to resource-aggregate-rework-rebase September 20, 2024 07:31
@PawelPawelec-RDX PawelPawelec-RDX marked this pull request as ready for review September 20, 2024 07:31
@PawelPawelec-RDX PawelPawelec-RDX changed the base branch from resource-aggregate-rework-rebase to develop September 20, 2024 08:43
@PawelPawelec-RDX PawelPawelec-RDX force-pushed the kv-store-query-alignment branch from b6b1fb7 to 6596ae4 Compare October 1, 2024 13:09
dhedey
dhedey approved these changes Oct 1, 2024
View reviewed changes
Copy link
Contributor

@dhedey dhedey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

KV Store query looks good 👍

src/RadixDlt.NetworkGateway.PostgresIntegration/Queries/KeyValueStoreQueries.cs Outdated Show resolved Hide resolved
src/RadixDlt.NetworkGateway.PostgresIntegration/Queries/KeyValueStoreQueries.cs Outdated Show resolved Hide resolved
src/RadixDlt.NetworkGateway.PostgresIntegration/Queries/KeyValueStoreQueries.cs Outdated Show resolved Hide resolved
src/RadixDlt.NetworkGateway.PostgresIntegration/Queries/KeyValueStoreQueries.cs Show resolved Hide resolved
src/RadixDlt.NetworkGateway.PostgresIntegration/Queries/KeyValueStoreQueries.cs Outdated Show resolved Hide resolved
src/RadixDlt.NetworkGateway.PostgresIntegration/Queries/KeyValueStoreQueries.cs Outdated Show resolved Hide resolved
@PawelPawelec-RDX PawelPawelec-RDX force-pushed the kv-store-query-alignment branch from 1470044 to a09393b Compare October 3, 2024 08:51
@PawelPawelec-RDX PawelPawelec-RDX force-pushed the kv-store-query-alignment branch from a09393b to adb322a Compare October 3, 2024 08:59
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Oct 3, 2024

Quality Gate Passed Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@PawelPawelec-RDX PawelPawelec-RDX merged commit 5110270 into develop Oct 3, 2024
15 of 16 checks passed
@PawelPawelec-RDX PawelPawelec-RDX deleted the kv-store-query-alignment branch October 3, 2024 13:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dhedey dhedey dhedey approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@PawelPawelec-RDX @dhedey