Forward cache bugfix index pages queue verify #3134
Conversation
|
⚪ |
|
⚪ |
| @@ -20,10 +20,10 @@ namespace NTable { | |||
| 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, чтобы можно было только работу со страницами/блобами замокать, а всё остальное оставить. Идея красивая, иметь возможность запускать локальную базу поверх чего угодно, не только блоб стораджа, но на практике скорее не нужно, и абстракция всё-равно протекает по факту. :-/
|
⚪
|
|
⚪
|
kunga
changed the title
| @@ -352,15 +353,15 @@ namespace NFwd { | |||
| } | |||
| } | |||
|
|
|||
| 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 и удалять блобы от скомпакченных данных). Но в целом я не против переименования.
| @@ -20,10 +20,10 @@ namespace NTable { | |||
| namespace { | |||
| using namespace NTest; | |||
|
|
|||
| struct TFramesWrap : public NTest::TSteps<TFramesWrap>, protected NFwd::IPageLoadingQueue { | |||
There was a problem hiding this comment.
Тут я соглашусь, я когда впервые в локальную базу пришёл тоже удивился - в тестах очень многие такие структуры повторяются, в итоге тестируется не код локальной базы, а продублированный код тестов. Видимо оно у нас недостаточно generic, чтобы можно было только работу со страницами/блобами замокать, а всё остальное оставить. Идея красивая, иметь возможность запускать локальную базу поверх чего угодно, не только блоб стораджа, но на практике скорее не нужно, и абстракция всё-равно протекает по факту. :-/
|
⚪
|
|
⚪ |
|
⚪
|
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)