Re: Parallel Hash take II

From: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
To: Rushabh Lathia <rushabh(dot)lathia(at)gmail(dot)com>
Cc: Peter Geoghegan <pg(at)bowt(dot)ie>, Robert Haas <robertmhaas(at)gmail(dot)com>, Prabhat Sahu <prabhat(dot)sahu(at)enterprisedb(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org>, Rafia Sabih <rafia(dot)sabih(at)enterprisedb(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>, Oleg Golovanov <rentech(at)mail(dot)ru>
Subject: Re: Parallel Hash take II
Date: 2017-10-05 07:45:58
Views: Raw Message | Whole Thread | Download mbox
Lists: pgsql-hackers

On Thu, Oct 5, 2017 at 7:07 PM, Rushabh Lathia <rushabh(dot)lathia(at)gmail(dot)com> wrote:
> v20 patch set (I was trying 0008, 0009 patch) not getting cleanly apply on
> latest commit also getting compilation error due to refactor in below
> commit.
> commit 0c5803b450e0cc29b3527df3f352e6f18a038cc6

Hi Rushabh

I am about to post a new patch set that fixes the deadlock problem,
but in the meantime here is a rebase of those two patches (numbers
changed to 0006 + 0007). In the next version I think I should
probably remove that 'stripe' concept. The idea was to spread
temporary files over the available temporary tablespaces, but it's a
terrible API, since you have to promise to use the same stripe number
when opening the same name later... Maybe I should use a hash of the
name for that instead. Thoughts?

Thomas Munro

Attachment Content-Type Size
0006-Remove-BufFile-s-isTemp-flag.patch application/octet-stream 3.9 KB
0007-Add-BufFileSet-for-sharing-temporary-files-between-b.patch application/octet-stream 37.2 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Etsuro Fujita 2017-10-05 09:08:50 Re: Another oddity in handling of WCO constraints in postgres_fdw
Previous Message Andres Freund 2017-10-05 06:57:39 Re: JIT compiling - v4.0