Skip to content

feat: sst_min_sequences in QueryContext#234

Merged
MichaelScofield merged 1 commit intomainfrom
feat/query-context-min-sequences
May 7, 2025
Merged

feat: sst_min_sequences in QueryContext#234
MichaelScofield merged 1 commit intomainfrom
feat/query-context-min-sequences

Conversation

@MichaelScofield
Copy link
Copy Markdown
Collaborator

@MichaelScofield MichaelScofield commented May 6, 2025

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

What's changed and what's your intention?

add a "minimal sequence" in query context message to filter out some unwanted SST files, useful in our "leader-read" feature

Checklist

  • I have written the necessary comments.
  • I have added the necessary unit tests and integration tests.

@MichaelScofield MichaelScofield requested a review from discord9 May 6, 2025 02:55
@MichaelScofield MichaelScofield force-pushed the feat/query-context-min-sequences branch 2 times, most recently from 02947a4 to 4c9653d Compare May 6, 2025 06:47
@MichaelScofield MichaelScofield changed the title feat: sequences in QueryContext feat: sst_min_sequences in QueryContext May 6, 2025
@MichaelScofield
Copy link
Copy Markdown
Collaborator Author

db change: GreptimeTeam/greptimedb#6051

@MichaelScofield MichaelScofield marked this pull request as ready for review May 6, 2025 07:39
@MichaelScofield MichaelScofield force-pushed the feat/query-context-min-sequences branch from 4c9653d to cf7e12d Compare May 6, 2025 07:55
@MichaelScofield MichaelScofield force-pushed the feat/query-context-min-sequences branch from cf7e12d to d32f9fe Compare May 7, 2025 01:47
@MichaelScofield MichaelScofield force-pushed the feat/query-context-min-sequences branch from d32f9fe to f58ef98 Compare May 7, 2025 03:04
@MichaelScofield MichaelScofield merged commit 159e92d into main May 7, 2025
6 checks passed
@MichaelScofield MichaelScofield deleted the feat/query-context-min-sequences branch May 7, 2025 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants