Skip to content

RFC: Concurrent Stat in List#3574

Merged
Xuanwo merged 6 commits intoapache:mainfrom
morristai:feat/rfc_for_concurrent_stat_in_list
Nov 15, 2023
Merged

RFC: Concurrent Stat in List#3574
Xuanwo merged 6 commits intoapache:mainfrom
morristai:feat/rfc_for_concurrent_stat_in_list

Conversation

@morristai
Copy link
Copy Markdown
Member

@morristai morristai commented Nov 13, 2023

Description

relate: #3097

@morristai morristai requested a review from Xuanwo as a code owner November 13, 2023 06:08
@github-actions github-actions Bot added the releases-note/feat The PR implements a new feature or has a title that begins with "feat" label Nov 13, 2023
@Xuanwo Xuanwo changed the title feat(RFC): add concurrent stat in list RFC: Concurrent Stat in List Nov 13, 2023
Comment thread core/src/docs/rfcs/3574_concurrent_stat_in_list.md
Comment thread core/src/docs/rfcs/3574_concurrent_stat_in_list.md Outdated
Comment thread core/src/docs/rfcs/3574_concurrent_stat_in_list.md Outdated
Comment thread core/src/docs/rfcs/3574_concurrent_stat_in_list.md Outdated
Comment thread core/src/docs/rfcs/3574_concurrent_stat_in_list.md Outdated
Comment thread core/src/docs/rfcs/3574_concurrent_stat_in_list.md Outdated
Comment thread core/src/docs/rfcs/3574_concurrent_stat_in_list.md Outdated
Copy link
Copy Markdown
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Mostly LGTM! Thanks for carrying the RFC.

Comment thread core/src/docs/rfcs/3574_concurrent_stat_in_list.md Outdated
@Xuanwo
Copy link
Copy Markdown
Member

Xuanwo commented Nov 15, 2023

Inviting more committers to review the RFC.

Comment thread core/src/docs/rfcs/3574_concurrent_stat_in_list.md
Copy link
Copy Markdown
Member

@oowl oowl left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks

Copy link
Copy Markdown
Member

@suyanhanx suyanhanx left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks.

@Xuanwo Xuanwo merged commit b4a2221 into apache:main Nov 15, 2023
Copy link
Copy Markdown
Member

@dqhl76 dqhl76 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@morristai morristai deleted the feat/rfc_for_concurrent_stat_in_list branch November 16, 2023 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

releases-note/feat The PR implements a new feature or has a title that begins with "feat"

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants