Forward cache bugfix index pages queue verify #3134
Conversation
|
⚪ |
|
⚪ |
| namespace { | ||
| using namespace NTest; | ||
|
|
||
| struct TFramesWrap : public NTest::TSteps<TFramesWrap>, protected NFwd::IPageLoadingQueue { |
There was a problem hiding this comment.
Хотел написать сюда честный тест на TEnv, но оказалось что тот TPartStore который создается TMake не кастуется к NTable::TPartStore
Поэтому пришлось писать тесты через Executor
Странная фигня, наверно надо бы это как-то исправить
There was a problem hiding this comment.
Тут я соглашусь, я когда впервые в локальную базу пришёл тоже удивился - в тестах очень многие такие структуры повторяются, в итоге тестируется не код локальной базы, а продублированный код тестов. Видимо оно у нас недостаточно generic, чтобы можно было только работу со страницами/блобами замокать, а всё остальное оставить. Идея красивая, иметь возможность запускать локальную базу поверх чего угодно, не только блоб стораджа, но на практике скорее не нужно, и абстракция всё-равно протекает по факту. :-/
|
⚪
|
|
⚪
|
kungasc
changed the title
| } | ||
|
|
||
| TEgg MakeCache(const TPart *part, NPage::TGroupId groupId, TIntrusiveConstPtr<TSlices> bounds) noexcept | ||
| TEgg MakeCache(const TPart *part, NPage::TGroupId groupId, TIntrusiveConstPtr<TSlices> slices) noexcept |
There was a problem hiding this comment.
Кстати оно в сканах называлось bounds, потому что я хотел подчеркнуть, что грузить что-либо за границами этих строк строго запрещено, там блобов может уже не существовать (шардированный компакшен умел компактить половину sst и удалять блобы от скомпакченных данных). Но в целом я не против переименования.
| namespace { | ||
| using namespace NTest; | ||
|
|
||
| struct TFramesWrap : public NTest::TSteps<TFramesWrap>, protected NFwd::IPageLoadingQueue { |
There was a problem hiding this comment.
Тут я соглашусь, я когда впервые в локальную базу пришёл тоже удивился - в тестах очень многие такие структуры повторяются, в итоге тестируется не код локальной базы, а продублированный код тестов. Видимо оно у нас недостаточно generic, чтобы можно было только работу со страницами/блобами замокать, а всё остальное оставить. Идея красивая, иметь возможность запускать локальную базу поверх чего угодно, не только блоб стораджа, но на практике скорее не нужно, и абстракция всё-равно протекает по факту. :-/
|
⚪
|
|
⚪ |
|
⚪
|
2 participants
Changelog entry
...
Changelog category
Additional information
Fix "Cache line head don't want to do fetch but should" verify: we may request index page from Queues[0] while handling another queue (for example a history one)