Re: Different compression methods for FPI

Поиск
Список
Период
Сортировка
От Andrey Borodin
Тема Re: Different compression methods for FPI
Дата
Msg-id 5C10FAD9-DC15-45E5-BCF9-B179A1ABA82E@yandex-team.ru
обсуждение исходный текст
Ответ на Re: Different compression methods for FPI  (Michael Paquier <michael@paquier.xyz>)
Ответы Re: Different compression methods for FPI  (Michael Paquier <michael@paquier.xyz>)
Список pgsql-hackers

> 17 июня 2021 г., в 11:44, Michael Paquier <michael@paquier.xyz> написал(а):
>
> I have worked more on that today and finished with two patches

I have some small questions.

1.
+            report_invalid_record(record, "image at %X/%X compressed with %s not supported, block %d",
+                                  (uint32) (record->ReadRecPtr >> 32),
+                                  (uint32) record->ReadRecPtr,
+                                  "lz4",
+                                  block_id);
Can we point out to user that the problem is in the build? Also, maybe %s can be inlined to lz4 in this case.

2.
> const char *method = "???";
Maybe we can use something like "unknown" for unknown compression methods? Or is it too long string for waldump output?

3. Can we exclude lz4 from config if it's not supported by the build?

Thanks!

Best regards, Andrey Borodin.




В списке pgsql-hackers по дате отправления:

Предыдущее
От: Andrew Dunstan
Дата:
Сообщение: Re: PXGS vs TAP tests
Следующее
От: Tom Lane
Дата:
Сообщение: Re: Race condition in InvalidateObsoleteReplicationSlots()