Skip to content

issue-6414: [Filestore] Get rid of square complexity of UpdateCounters#6543

Open
ruslan-gar wants to merge 5 commits into
mainfrom
users/ruslan-gar/get_rid_of_square_complexity_of_upadte_counters
Open

issue-6414: [Filestore] Get rid of square complexity of UpdateCounters#6543
ruslan-gar wants to merge 5 commits into
mainfrom
users/ruslan-gar/get_rid_of_square_complexity_of_upadte_counters

Conversation

@ruslan-gar

Copy link
Copy Markdown
Collaborator

Notes

When TAggregateStatsActor collects statistics for a shard, it doesn't send requests to all shards but instead requests
cached statistics from the main tablet.

Issue

#6414

@ruslan-gar
ruslan-gar requested a review from debnatkh July 16, 2026 17:04
@ruslan-gar ruslan-gar added filestore Add this label to run only cloud/filestore build and tests on PR large-tests Launch large tests for PR labels Jul 16, 2026
@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Note

This is an automated comment that will be appended during run.

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

🟢 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 6707s): all tests PASSED for commit dd56c2e.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
4232 4232 0 0 0 0 0

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: dd56c2e572

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp Outdated
Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp
Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp Outdated
@github-actions

github-actions Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Note

This is an automated comment that will be appended during run.

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

🟢 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 6703s): all tests PASSED for commit 0fe6993.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
4232 4232 0 0 0 0 0

Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp Outdated
Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp
Comment thread cloud/filestore/private/api/protos/tablet.proto Outdated
Comment thread cloud/filestore/libs/storage/service/service_ut_sharding.cpp Outdated
Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp Outdated
Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp Outdated
Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp Outdated
Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp Outdated
Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp Outdated
Comment thread cloud/filestore/libs/storage/tablet/tablet_actor_counters.cpp Outdated
@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Note

This is an automated comment that will be appended during run.

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Note

This is an automated comment that will be appended during run.

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

🟢 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 7184s): all tests PASSED for commit b30c15a.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
4233 4233 0 0 0 0 0

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Note

This is an automated comment that will be appended during run.

Note

All workloads for linux-x86_64-relwithdebinfo have completed.

Tip

Planned checks for linux-x86_64-relwithdebinfo.

🟢 linux-x86_64-relwithdebinfo target: cloud/filestore/ (test time: 7033s): all tests PASSED for commit f16de78.

TESTS PASSED ERRORS FAILED FAILED BUILD SKIPPED MUTED?
4392 4392 0 0 0 0 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

filestore Add this label to run only cloud/filestore build and tests on PR large-tests Launch large tests for PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants