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

issue-1794: fixed use-after-free in fs search by alias, not making a new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession #1816

Merged
merged 2 commits into from
Aug 18, 2024

Conversation

…new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession
@qkrorlqr qkrorlqr added filestore Add this label to run only cloud/filestore build and tests on PR asan Launch builds with address sanitizer along with regular build labels Aug 18, 2024
…new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession - fixed comment
Copy link
Contributor

Note

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

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 24a3958.

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

Copy link
Contributor

Note

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

🟢 linux-x86_64-relwithdebinfo: all tests PASSED for commit 24a3958.

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

🟢 linux-x86_64-release-asan: all tests PASSED for commit 24a3958.

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

@qkrorlqr qkrorlqr merged commit 449ede9 into main Aug 18, 2024
14 of 15 checks passed
@qkrorlqr qkrorlqr deleted the users/qkrorlqr/issue-1794-fixes branch August 18, 2024 11:01
debnatkh pushed a commit that referenced this pull request Aug 18, 2024
…new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession (#1816)

* issue-1794: fixed use-after-free in fs search by alias, not making a new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession

* issue-1794: fixed use-after-free in fs search by alias, not making a new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession - fixed comment
debnatkh added a commit that referenced this pull request Aug 19, 2024
…ompaction stat ranges; add action to write specified data to compaction map table, nfs-client.txt, support aliases for filestore names (#1817)

* add force operation to delete all zero compaction stat ranges; add aсtion to write specified data to compaction map table (#1690)

* add force operation to delete all zero compaction stat ranges; add action to write specified data to compaction map table

* fix-issues

* fix issues

* fix

* [Filestore] add `nfs-client.txt` config for filestore analogous to the `nbs-client.txt` one (#1687)

add nfs-client.txt config for filestore analogous to the nbs-client one

* issue-1794: support aliases for filestore names (#1774)

* issue-1794: support aliases for filestore names

* fix cmakelists

* issue-1794: fixed use-after-free in fs search by alias, not making a new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession (#1816)

* issue-1794: fixed use-after-free in fs search by alias, not making a new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession

* issue-1794: fixed use-after-free in fs search by alias, not making a new THashMap upon each alias search, removed incorrect TABLET_VERIFY in TIndexTabletActor::CompleteTx_CreateSession - fixed comment

* fix build: exclude TCertificate from config.cpp

* fix build

* fix build

* fix build

---------

Co-authored-by: Darya Frolova <47457802+WilyTiger@users.noreply.github.com>
Co-authored-by: Darya Frolova <wilytiger@nebius.com>
Co-authored-by: Andrei Strelkovskii <thinkingwhat@yandex.ru>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
asan Launch builds with address sanitizer along with regular build filestore Add this label to run only cloud/filestore build and tests on PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants