Skip to content

Conversation

v0y4g3r
Copy link
Contributor

@v0y4g3r v0y4g3r commented Mar 15, 2023

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

This PR allows users to manually flush table through HTTP API.

# flush region 1 of table greptime.benchmark.tab
curl -XPOST 'localhost:4000/v1/admin/flush?catalog_name=greptime&schema_name=benchmark&table_name=tab&region=1'

# flush all regions of table greptime.benchmark.tab
curl -XPOST 'localhost:4000/v1/admin/flush?catalog_name=greptime&schema_name=benchmark&table_name=tab'

# flush all tables of table greptime.benchmark
curl -XPOST 'localhost:4000/v1/admin/flush?catalog_name=greptime&schema_name=benchmark'

Checklist

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

Refer to a related PR or issue link (optional)

@codecov
Copy link

codecov bot commented Mar 15, 2023

Codecov Report

Merging #1184 (7caa6b0) into develop (cd8be77) will decrease coverage by 0.19%.
The diff coverage is 45.60%.

@@             Coverage Diff             @@
##           develop    #1184      +/-   ##
===========================================
- Coverage    85.21%   85.03%   -0.19%     
===========================================
  Files          484      485       +1     
  Lines        71798    71903     +105     
===========================================
- Hits         61186    61143      -43     
- Misses       10612    10760     +148     

@v0y4g3r v0y4g3r requested review from killme2008, waynexia and evenyag and removed request for killme2008 March 15, 2023 10:17
Copy link
Member

@waynexia waynexia left a comment

Choose a reason for hiding this comment

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

LGTM

@waynexia waynexia enabled auto-merge (squash) March 15, 2023 12:14
Copy link
Contributor

@evenyag evenyag left a comment

Choose a reason for hiding this comment

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

LGTM

@waynexia waynexia merged commit 17eb99b into GreptimeTeam:develop Mar 15, 2023
@evenyag evenyag mentioned this pull request Apr 25, 2023
paomian pushed a commit to paomian/greptimedb that referenced this pull request Oct 19, 2023
@2547409617
Copy link

Hi guys,
I'm a newbie to github. I am very interested in this feature, but I can't find the committed code in develop branch.
Any suggestions or tips are very welcome.
Thanks.

@MichaelScofield
Copy link
Collaborator

Seems it's removed in #2285
Any clues? @fengjiachun

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.

5 participants