Horton

Thread

Commits

  1. Mark assorted GUC variables as PGDLLIMPORT.

  1. Add PGDLLIMPORT to enable_hashagg

    Metin Doslu <metin@citusdata.com> — 2018-02-05T11:17:43Z

    Hey all,

There was already a discussion and commit for adding PGDLLIMPORT to some
variables which enables extensions to use them on Windows builds. For
reference, the previous thread:"Add PGDLLIMPORT lines to some variables".

I would like to add PGDLLIMPORT to enable_hashagg for the same reason. I'm
adding a very simple patch. Please let me know if I'm missing anything
given that this is my first patch submission.

Best,
Metin

  2. Re: Add PGDLLIMPORT to enable_hashagg

    Robert Haas <robertmhaas@gmail.com> — 2018-02-06T20:17:50Z

    On Mon, Feb 5, 2018 at 6:17 AM, Metin Doslu <metin@citusdata.com> wrote:
> There was already a discussion and commit for adding PGDLLIMPORT to some
> variables which enables extensions to use them on Windows builds. For
> reference, the previous thread:"Add PGDLLIMPORT lines to some variables".
>
> I would like to add PGDLLIMPORT to enable_hashagg for the same reason. I'm
> adding a very simple patch. Please let me know if I'm missing anything given
> that this is my first patch submission.

Seems weird to do it just for enable_hashagg and not for all of the
surrounding Booleans.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

  3. Re: Add PGDLLIMPORT to enable_hashagg

    legrand legrand <legrand_legrand@hotmail.com> — 2018-02-06T20:34:10Z

    Hello,
I know an other extension that would need the same for:

src/include/optimizer/paths.h
geqo_threshold
enable_geqo
min_parallel_index_scan_size
min_parallel_table_scan_size

src/include/optimizer/cost.h
max_parallel_workers_per_gather
enable_hashjoin
enable_mergejoin
enable_nestloop
enable_indexonlyscan
enable_tidscan
enable_bitmapscan
enable_indexscan
enable_seqscan
disable_cost

src/include/utils/guc.h
application_name

src/include/miscadmin.h
max_worker_processes

see https://github.com/ossc-db/pg_hint_plan/issues/8

Regards
PAscal



--
Sent from: http://www.postgresql-archive.org/PostgreSQL-hackers-f1928748.html

  4. Re: Add PGDLLIMPORT to enable_hashagg

    Robert Haas <robertmhaas@gmail.com> — 2018-02-06T20:39:41Z

    On Tue, Feb 6, 2018 at 3:34 PM, legrand legrand
<legrand_legrand@hotmail.com> wrote:
> Hello,
> I know an other extension that would need the same for:
>
> src/include/optimizer/paths.h
> geqo_threshold
> enable_geqo
> min_parallel_index_scan_size
> min_parallel_table_scan_size
>
> src/include/optimizer/cost.h
> max_parallel_workers_per_gather
> enable_hashjoin
> enable_mergejoin
> enable_nestloop
> enable_indexonlyscan
> enable_tidscan
> enable_bitmapscan
> enable_indexscan
> enable_seqscan
> disable_cost
>
> src/include/utils/guc.h
> application_name
>
> src/include/miscadmin.h
> max_worker_processes
>
> see https://github.com/ossc-db/pg_hint_plan/issues/8

Yeah, let's get them all into one list and I'll commit the whole thing together.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

  5. Re: Add PGDLLIMPORT to enable_hashagg

    Peter Geoghegan <pg@bowt.ie> — 2018-02-06T20:40:41Z

    On Tue, Feb 6, 2018 at 12:39 PM, Robert Haas <robertmhaas@gmail.com> wrote:
> Yeah, let's get them all into one list and I'll commit the whole thing together.

+1

-- 
Peter Geoghegan

  6. Re: Add PGDLLIMPORT to enable_hashagg

    Metin Doslu <metin@citusdata.com> — 2018-02-07T11:32:12Z

    Hey all,

I'm attaching the updated patch, it includes

i. The list of Pascal (max_worker_processes was already with
PGDLLIMPORT, so I also added to max_parallel_workers)
ii. Some others in cost.h to make the file more readable.

Best,
Metin

On Tue, Feb 6, 2018 at 10:40 PM, Peter Geoghegan <pg@bowt.ie> wrote:
> On Tue, Feb 6, 2018 at 12:39 PM, Robert Haas <robertmhaas@gmail.com> wrote:
>> Yeah, let's get them all into one list and I'll commit the whole thing together.
>
> +1
>
> --
> Peter Geoghegan
>

  7. Re: Add PGDLLIMPORT to enable_hashagg

    legrand legrand <legrand_legrand@hotmail.com> — 2018-02-07T21:13:29Z

    Thank you Metin !

Regards
PAscal



--
Sent from: http://www.postgresql-archive.org/PostgreSQL-hackers-f1928748.html

  8. Re: Add PGDLLIMPORT to enable_hashagg

    Robert Haas <robertmhaas@gmail.com> — 2018-02-09T21:01:35Z

    On Wed, Feb 7, 2018 at 6:32 AM, Metin Doslu <metin@citusdata.com> wrote:
> i. The list of Pascal (max_worker_processes was already with
> PGDLLIMPORT, so I also added to max_parallel_workers)
> ii. Some others in cost.h to make the file more readable.

Committed.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

  9. Re: Add PGDLLIMPORT to enable_hashagg

    Brian Cloutier <brian@citusdata.com> — 2018-02-16T20:57:40Z

    On Fri, Feb 9, 2018 at 1:01 PM, Robert Haas <robertmhaas@gmail.com> wrote:

>
> Committed.


Thanks for committing this! We forgot to ask though, could you please
backport this patch to 10 and maybe even 9.6? As-is I don't think these
variables will be available until PG 11.

  10. Re: Add PGDLLIMPORT to enable_hashagg

    Andres Freund <andres@anarazel.de> — 2018-02-21T18:14:06Z

    Hi,

On 2018-02-16 12:57:40 -0800, Brian Cloutier wrote:
> Thanks for committing this! We forgot to ask though, could you please
> backport this patch to 10 and maybe even 9.6? As-is I don't think these
> variables will be available until PG 11.

I think these days our policy is to do that on request, which e.g. was
done on c572599c. Could you take the relevant commit, backport it to the
relevant branches, resolve conflicts, make possibly appropriate
adaptions, and post?

Greetings,

Andres Freund

  11. Re: Add PGDLLIMPORT to enable_hashagg

    Brian Cloutier <brian@citusdata.com> — 2018-02-21T19:41:31Z

    On Wed, Feb 21, 2018 at 10:14 AM, Andres Freund <andres@anarazel.de> wrote:

> Could you take the relevant commit, backport it to the
> relevant branches, resolve conflicts, make possibly appropriate
> adaptions, and post?
>

The original commit touched some new variables and therefore didn't apply
cleanly. Attached are equivalent patches for REL_10_STABLE and
REL9_6_STABLE.

  12. Re: Add PGDLLIMPORT to enable_hashagg

    Andres Freund <andres@anarazel.de> — 2018-02-22T21:04:46Z

    On 2018-02-21 11:41:31 -0800, Brian Cloutier wrote:
> On Wed, Feb 21, 2018 at 10:14 AM, Andres Freund <andres@anarazel.de> wrote:
> 
> > Could you take the relevant commit, backport it to the
> > relevant branches, resolve conflicts, make possibly appropriate
> > adaptions, and post?
> >
> 
> The original commit touched some new variables and therefore didn't apply
> cleanly. Attached are equivalent patches for REL_10_STABLE and
> REL9_6_STABLE.

Pushed.

- Andres