Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

merge to stable-23-3: filestore: leader->shard ResetSession, refactoring, fixes in filestore-client, fix in BlobsCount accounting; minor fixes in blockstore #2287

Merged
merged 14 commits into from
Oct 16, 2024

Conversation

qkrorlqr
Copy link
Collaborator

…n StorageService and accessed by the code calculating MaxTime for inflight requests at the same time - we should not reuse the same CallContext for different requests, we should make separate CallContexts (#2223)
… cleanup (#2225)

* filestore-client findgarbage paging fix

* filestore-client - outputting garbage file sizes in the findgarbage command output; tablet - cleanup: MaxResponseBytes config param was unused, using it

* filestore-client - outputting garbage file sizes in the findgarbage command output; tablet - cleanup: MaxResponseBytes config param was unused, using it - recanon test
…data_stress (#2236)

* moved ShouldHandleRangeIdCollisionsInCompactionMapStats to tablet_ut_data_stress

* moved ShouldHandleRangeIdCollisionsInCompactionMapStats to tablet_ut_data_stress - fix
…lready calls those entities "shards" 2. "follower" implies some kind of replication and this entity is not related to replication at all (#2246)

* issue-1932: renamed follower to shard, reasons: 1. some of the code already calls those entities shards 2. follower implies some kind of replication and this entity is not related to replication at all

* issue-1932: renamed follower to shard, reasons: 1. some of the code already calls those entities shards 2. follower implies some kind of replication and this entity is not related to replication at all - fix build

* issue-1932: renamed follower to shard, reasons: 1. some of the code already calls those entities shards 2. follower implies some kind of replication and this entity is not related to replication at all - fix canondata

* issue-1932: renamed follower to shard, reasons: 1. some of the code already calls those entities shards 2. follower implies some kind of replication and this entity is not related to replication at all - fix canondata
…he Session-aware methods to properly process E_FS_INVALID_SESSION errors; existing tests are more or less sufficient to ensure that this code is correct (#2265)
… to leader (in the same way it's done for Create/DestroySession); extracted common code for this thing from Create/DestroySession (#2266)

* issue-2033: sending ResetSession requests to shards upon ResetSession to leader (in the same way it's done for Create/DestroySession); extracted common code for this thing from Create/DestroySession

* issue-2033: fixed logging

* issue-2033: better

* issue-2033: cleanup
yegorskii
yegorskii previously approved these changes Oct 16, 2024
@qkrorlqr qkrorlqr added the rebase Add this label if you want to rebase your PR for test run label Oct 16, 2024
debnatkh
debnatkh previously approved these changes Oct 16, 2024
yegorskii
yegorskii previously approved these changes Oct 16, 2024
@qkrorlqr qkrorlqr dismissed stale reviews from yegorskii and debnatkh via 44f19c3 October 16, 2024 11:20
Copy link
Contributor

Note

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

🟢 default-linux-x86-64-relwithdebinfo: all tests PASSED for commit 44f19c3.

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
1551 1551 0 0 0 0

@debnatkh debnatkh merged commit 3289b84 into stable-23-3 Oct 16, 2024
6 of 9 checks passed
@debnatkh debnatkh deleted the users/qkrorlqr/stable-23-3 branch October 16, 2024 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rebase Add this label if you want to rebase your PR for test run
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants