Skip to content

docs: rfc of update metadata in single txn#2165

Merged
v0y4g3r merged 2 commits intoGreptimeTeam:developfrom
WenyXu:rfcs/metadata-txn
Aug 15, 2023
Merged

docs: rfc of update metadata in single txn#2165
v0y4g3r merged 2 commits intoGreptimeTeam:developfrom
WenyXu:rfcs/metadata-txn

Conversation

@WenyXu
Copy link
Copy Markdown
Member

@WenyXu WenyXu commented Aug 13, 2023

I hereby agree to the terms of the GreptimeDB CLA

What's changed and what's your intention?

The RFC of update metadata in single txn

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)

#1715

@WenyXu WenyXu force-pushed the rfcs/metadata-txn branch from 7fc4c63 to 0072fa6 Compare August 13, 2023 12:42
Comment thread docs/rfcs/2023-08-13-metadata-txn.md
Comment thread docs/rfcs/2023-08-13-metadata-txn.md Outdated
@WenyXu WenyXu force-pushed the rfcs/metadata-txn branch 2 times, most recently from d11c8c5 to 3c25171 Compare August 14, 2023 14:05
@WenyXu
Copy link
Copy Markdown
Member Author

WenyXu commented Aug 15, 2023

@MichaelScofield PTAL

@WenyXu WenyXu force-pushed the rfcs/metadata-txn branch from 3c25171 to fac6986 Compare August 15, 2023 03:59
@fengjiachun fengjiachun disabled auto-merge August 15, 2023 06:17
@fengjiachun fengjiachun enabled auto-merge August 15, 2023 06:17
@WenyXu WenyXu disabled auto-merge August 15, 2023 06:18
@v0y4g3r v0y4g3r merged commit f5e44ba into GreptimeTeam:develop Aug 15, 2023
MichaelScofield pushed a commit to MichaelScofield/greptimedb that referenced this pull request Aug 16, 2023
* feat: handle multiple grpc deletes (GreptimeTeam#2150)

* feat: handle multiple grpc deletes

* fix: make DistDeleter::grpc_delete return usize

* fix: remove backtrace from MissingTimeIndexColumn

* fix: avoid using unwrap in PartitionRuleManager::split_delete_request

* fix: simplify MissingTimeIndexColumn

* refactor: improve the image building performance (GreptimeTeam#2175)

* refactor: use '--output type=local' in 'build-greptime-by-buildx' target to reduce unnecessary 'docker cp'"

Signed-off-by: zyy17 <zyylsxm@gmail.com>

* refactor: improve the image building performance

* ci: release centos dev builder

* ci: use 'make build-by-dev-builder' to improve docker build performance

* refactor: add 'which' command in centos

* fix: add 'OUTPUT_DIR' to fix 'make docker-image-buildx' error

* fix: fix incorrect dockerfile path

Signed-off-by: zyy17 <zyylsxm@gmail.com>

* refactor: remove configure-aws-credentials action and use env variables

Signed-off-by: zyy17 <zyylsxm@gmail.com>

* ci: update slack notification prompt

* refactor: clean up the target directory before building artifacts of centos7

---------

Signed-off-by: zyy17 <zyylsxm@gmail.com>

* docs: rfc of update metadata in single txn (GreptimeTeam#2165)

* docs: rfc of update metadata in single txn

* chore: apply suggestion from CR

* feat: grpc handler result (GreptimeTeam#2107)

* feat: grpc handler inner result

* feat: ext header, x-greptime-err-code, x-greptime-err-msg

* fix: sqlness case

* chore: by comment

* fix: convert status to Error

* chore: minor `auth` crate change (GreptimeTeam#2176)

* chore: pub auth_mysql

* chore: pub all error

* chore: remove back to error

* chore: wrap failed permission check result to err

* chore: minor change

* feat: don't enable telemetry for debug building (GreptimeTeam#2177)

---------

Signed-off-by: zyy17 <zyylsxm@gmail.com>
Co-authored-by: Niwaka <61189782+NiwakaDev@users.noreply.github.com>
Co-authored-by: zyy17 <zyylsxm@gmail.com>
Co-authored-by: Weny Xu <wenymedia@gmail.com>
Co-authored-by: JeremyHi <jiachun_feng@proton.me>
Co-authored-by: shuiyisong <113876041+shuiyisong@users.noreply.github.com>
Co-authored-by: dennis zhuang <killme2008@gmail.com>
paomian pushed a commit to paomian/greptimedb that referenced this pull request Oct 19, 2023
* docs: rfc of update metadata in single txn

* chore: apply suggestion from CR
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.

4 participants