Hi! I'm reviewing code at toast_64bit_v2. Michael, isn't there a typo? static const toast_external_info toast_external_infos[TOAST_EXTERNAL_INFO_SIZE] = { [VARTAG_ONDISK_INT8] = { .toast_pointer_size = TOAST_POINTER_INT8_SIZE, .maximum_chunk_size = TOAST_MAX_CHUNK_SIZE_INT8, .to_external_data = ondisk_int8_to_external_data, .create_external_data = ondisk_int8_create_external_data, }, [VARTAG_ONDISK_OID] = { .toast_pointer_size = TOAST_POINTER_INT8_SIZE, <--- here .maximum_chunk_size = TOAST_MAX_CHUNK_SIZE_OID, .to_external_data = ondisk_oid_to_external_data, .create_external_data = ondisk_oid_create_external_data, }, };
Shouldn't TOAST_POINTER_INT8_SIZE be replaced with TOAST_POINTER_OID_SIZE? Regards, -- Nikita Malakhov Postgres Professional The Russian Postgres Company https://postgrespro.ru/