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

[CDAP-21079] Update primary key sequencing for better indexing in Spanner Messaging Service #15795

Merged
merged 1 commit into from
Jan 9, 2025
Merged

[CDAP-21079] Update primary key sequencing for better indexing in Spanner Messaging Service #15795

merged 1 commit into from
Jan 9, 2025

Conversation

sidhdirenge
Copy link
Contributor

@sidhdirenge sidhdirenge commented Jan 9, 2025 •
edited
Loading

publish_ts should come first so we can use index efficiently.

tested in junit

image

More experiments related to performance are needed

@sidhdirenge sidhdirenge added the build Triggers github actions build label Jan 9, 2025
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Jan 9, 2025

Quality Gate Failed Quality Gate failed

Failed conditions
0.0% Coverage on New Code (required ≥ 80%)
E Reliability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@sidhdirenge sidhdirenge merged commit 9e317c1 into develop Jan 9, 2025
13 of 14 checks passed
@sidhdirenge sidhdirenge deleted the sidhdirenge-patch-2 branch January 9, 2025 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@itsankit-google itsankit-google itsankit-google approved these changes

Assignees
No one assigned
Labels
build Triggers github actions build
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sidhdirenge @itsankit-google