Skip to content

fix: add fast_drop_path for droprequest#218

Merged
WenyXu merged 5 commits intoGreptimeTeam:mainfrom
yihong0618:main
Feb 25, 2025
Merged

fix: add fast_drop_path for droprequest#218
WenyXu merged 5 commits intoGreptimeTeam:mainfrom
yihong0618:main

Conversation

@yihong0618
Copy link
Copy Markdown
Contributor

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

for GreptimeTeam/greptimedb#5561

What's changed and what's your intention?

!!! DO NOT LEAVE THIS BLOCK EMPTY !!!

Please explain IN DETAIL what the changes are in this PR and why they are needed:

  • Summarize your change (mandatory)
  • How does this PR work? Need a brief introduction for the changed logic (optional)
  • Describe clearly one logical change and avoid lazy messages (optional)
  • Describe any limitations of the current code (optional)

Checklist

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

Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Comment thread proto/greptime/v1/region/server.proto Outdated
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Copy link
Copy Markdown
Contributor

@fengjiachun fengjiachun left a comment

Choose a reason for hiding this comment

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

LGTM

@yihong0618 yihong0618 changed the title fix: add fast_drop_database_path for droprequest fix: add fast_drop_path for droprequest Feb 20, 2025
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Copy link
Copy Markdown
Member

@WenyXu WenyXu left a comment

Choose a reason for hiding this comment

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

Rest LGTM

Comment thread proto/greptime/v1/region/server.proto Outdated
Signed-off-by: yihong0618 <zouzou0208@gmail.com>
Co-authored-by: WenyXu <wenymedia@gmail.com>
@WenyXu WenyXu enabled auto-merge (squash) February 25, 2025 08:35
@WenyXu WenyXu merged commit 072ce58 into GreptimeTeam:main Feb 25, 2025
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