Skip to content

Conversation

@drmingdrmer
Copy link
Member

@drmingdrmer drmingdrmer commented Sep 4, 2025

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

feat(meta-semaphore): allows to specify timestamp as semaphore seq

This is useful when retrying to acquire a semaphore:
the retry should use the previous timestamp otherwise it may never be
able to acquire

refactor: meta-semaphore: add logging

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

Type of change

  • Refactoring

Related Issues


This change is Reviewable

This is useful when retrying to acquire a semaphore:
the retry should use the previous timestamp otherwise it may never be
able to acquire
@drmingdrmer drmingdrmer marked this pull request as ready for review September 4, 2025 14:23
@github-actions github-actions bot added the pr-feature this PR introduces a new feature to the codebase label Sep 4, 2025
@zhang2014 zhang2014 merged commit 0912b80 into databendlabs:main Sep 5, 2025
95 checks passed
@drmingdrmer drmingdrmer deleted the split-semaphore branch September 5, 2025 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-feature this PR introduces a new feature to the codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants