Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-10 Thread Matthias van de Meent
On Tue, 9 Dec 2025 at 21:08, Hannu Krosing wrote: > > On Tue, Dec 9, 2025 at 7:41 PM Matthias van de Meent > wrote: > > > > On Mon, 8 Dec 2025 at 16:25, Hannu Krosing wrote: > > > Suggestion to "get the user specify their own identity column" sounds > > > good in purely theoretical sense, but ca

Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-09 Thread Hannu Krosing
On Tue, Dec 9, 2025 at 7:41 PM Matthias van de Meent wrote: > > On Mon, 8 Dec 2025 at 16:25, Hannu Krosing wrote: > > > ... > > Suggestion to "get the user specify their own identity column" sounds > > good in purely theoretical sense, but can have unacceptable overheads > > in practice, especial

Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-09 Thread Matthias van de Meent
On Mon, 8 Dec 2025 at 16:25, Hannu Krosing wrote: > > On Mon, Dec 8, 2025 at 2:46 PM Matthias van de Meent > wrote: > > I don't see logical decoding as a viable alternative to any > > indexing-related workload. Creating and maintaining indexes need > > strict limits on their resource usage, and l

Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-08 Thread Hannu Krosing
On Mon, Dec 8, 2025 at 2:46 PM Matthias van de Meent wrote: > > On Fri, 5 Dec 2025 at 16:50, Hannu Krosing wrote: > > > > On Fri, Dec 5, 2025 at 3:58 PM Euler Taveira wrote: > > > > > > On Thu, Dec 4, 2025, at 5:58 PM, Hannu Krosing wrote: > > > > Please find attached a patch that makes tuple id

Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-08 Thread Matthias van de Meent
On Fri, 5 Dec 2025 at 16:50, Hannu Krosing wrote: > > On Fri, Dec 5, 2025 at 3:58 PM Euler Taveira wrote: > > > > On Thu, Dec 4, 2025, at 5:58 PM, Hannu Krosing wrote: > > > Please find attached a patch that makes tuple ids and info about > > > weather it was plain or HOT update available to logi

Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-08 Thread Matthias van de Meent
On Thu, 4 Dec 2025 at 21:58, Hannu Krosing wrote: > > Please find attached a patch that makes tuple ids and info about > weather it was plain or HOT update available to logical decoding > callbacks. > > Also modified test_decoding to show both tids - > - old tid has format -(pageno, slot) > - new

Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-05 Thread Hannu Krosing
On Fri, Dec 5, 2025 at 3:58 PM Euler Taveira wrote: > > On Thu, Dec 4, 2025, at 5:58 PM, Hannu Krosing wrote: > > Please find attached a patch that makes tuple ids and info about > > weather it was plain or HOT update available to logical decoding > > callbacks. > > > > My first impression was: wh

Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-05 Thread Euler Taveira
On Thu, Dec 4, 2025, at 5:58 PM, Hannu Krosing wrote: > Please find attached a patch that makes tuple ids and info about > weather it was plain or HOT update available to logical decoding > callbacks. > My first impression was: why do you want to expose an internal information that is mostly usele

Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-05 Thread Hannu Krosing
On Thu, Dec 4, 2025 at 11:35 PM Mihail Nikalayeu wrote: > > Hello! > > Thanks for the patch. Thanks for reviewing :) > Few comments: > > 1) tests are failing, expected output files need to be updated Yes, I didn't want to start changing tests before other parts stabilized a bit. > 2) > > * Tre

Re: making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-04 Thread Mihail Nikalayeu
Hello! Thanks for the patch. Few comments: 1) tests are failing, expected output files need to be updated 2) > * Treat HOT update as normal updates. There is no useful > * information in the fact that we could make it a HOT update > * locally and the WAL layout is compatible. I think it feels

making tid and HOTness of UPDATE available to logical decoding plugins

2025-12-04 Thread Hannu Krosing
Please find attached a patch that makes tuple ids and info about weather it was plain or HOT update available to logical decoding callbacks. Also modified test_decoding to show both tids - - old tid has format -(pageno, slot) - new tid has format +(pageno, slot) if it is a HOT update, it is decode