Re: [HACKERS] Parallel Plans and Cost of non-filter functions

2017-11-04 Thread Tels
t for me: https://www.postgresql.org/message-id/28621.1509750807%40sss.pgh.pa.us (shortened: http://bit.ly/2zetO5T) Seems the email-obfuskation breaks such links? Here is a short-link for people reading it via the archive on http: http://bit.ly/2hF4lIt Best regards, Tels -- Sent via

Re: [HACKERS] Account for cost and selectivity of HAVING quals

2017-11-01 Thread Tels
Hello David, On Tue, October 31, 2017 7:54 pm, David G. Johnston wrote: > On Tue, Oct 31, 2017 at 4:31 PM, Tels <nospam-pg-ab...@bloodgate.com> > wrote: > >> >> ​​ >> That looks odd to me, it first uses output_tuples in a formula, then >> overwrites the val

Re: [HACKERS] Account for cost and selectivity of HAVING quals

2017-10-31 Thread Tels
ses output_tuples in a formula, then overwrites the value with a new value. Should these lines be swapped? Best regards, Tels -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] PL_stashcache, or, what's our minimum Perl version?

2017-08-01 Thread Tels
On Sun, July 30, 2017 4:35 pm, Tom Lane wrote: > "Tels" <nospam-pg-ab...@bloodgate.com> writes: >> On Sun, July 30, 2017 12:22 pm, Tom Lane wrote: >>> Yeah, I looked into that. The closest candidate I can find is that >>> perl 5.10.1 contains Tes

Re: [HACKERS] PL_stashcache, or, what's our minimum Perl version?

2017-07-30 Thread Tels
Moin, On Sun, July 30, 2017 12:22 pm, Tom Lane wrote: > "Tels" <nospam-pg-ab...@bloodgate.com> writes: >> On Sun, July 30, 2017 1:21 am, Tom Lane wrote: >>>> So the question is, does anyone care? I wouldn't except that our >>>> documentation appe

Re: [HACKERS] PL_stashcache, or, what's our minimum Perl version?

2017-07-30 Thread Tels
I do so happen to have a large archive with Perl releases and CPAN modules. It was first mirrored on mid-2015 - so anything that was deleted before 2015 unfortunately I can't help you with that. But if you need a specific module version, just ping me and I can see if it's in there. Hope this helps,

Re: [HACKERS] TAP: allow overriding PostgresNode in get_new_node

2017-06-01 Thread Tels
te to MyBase's code. While technically you can work around that by "peeking" into MyBase's code and maybe some reblessing, the point is that you shouldn't do nor need to do this. Please SEE: http://perldoc.perl.org/perlobj.html Regards, Tels -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] [bug fix] PG10: libpq doesn't connect to alternative hosts when some errors occur

2017-05-19 Thread Tels
, B is always skipped if it appears before A or C. The admin would see on the monitoring that B is offline (briefly or permanent) and need to correct it. >From the user's perspective, the second variant is smooth, the first is breaking randomly. A "database user" would not

Re: [HACKERS] [bug fix] PG10: libpq doesn't connect to alternative hosts when some errors occur

2017-05-17 Thread Tels
ts for you. However, if I understand it correctly, #3 only works reliable in certain cases (e.g. host down), but not if it is "sort of down". In that case each app would again need code to retry different hosts until it finds a working one, instead of letting libpq do the work. Tha

Re: [HACKERS] PG 10 release notes

2017-04-25 Thread Tels
On Tue, April 25, 2017 1:21 pm, Bruce Momjian wrote: > On Tue, Apr 25, 2017 at 01:06:05PM -0400, Tels wrote: >> Moin, >> >> On Mon, April 24, 2017 9:31 pm, Bruce Momjian wrote: >> > I have committed the first draft of the Postgres 10 release notes. >> They

Re: [HACKERS] PG 10 release notes

2017-04-25 Thread Tels
d not displayed in other cases. The new EXPLAIN option SUMMARY allows explicit control of this feature." Regards, Tels -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] WITH clause in CREATE STATISTICS

2017-04-21 Thread Tels
Moin, On Fri, April 21, 2017 7:04 am, David Rowley wrote: > On 21 April 2017 at 22:30, Tels <nospam-pg-ab...@bloodgate.com> wrote: >> I'd rather see: >> >> CREATE STATISTICS stats_name ON table(col); >> >> as this both mirrors CREATE INDEX and foreign

Re: [HACKERS] WITH clause in CREATE STATISTICS

2017-04-21 Thread Tels
so be extended to both more columns, expressions or other tables like so: CREATE STATISTICS stats ON t1(col1, col2 / 2), t2 (a,b); and even: CREATE STATISTICS stats ON t1(col1, col2 / 2), t2 (a,b) WITH (options) WHERE t2.a > 4; This looks easy to remember, since it compares to: CREATE INDEX

Re: [HACKERS] multivariate statistics (v25)

2017-04-05 Thread Tels
l_b) FROM table;" These do it the other way round: CREATE INDEX idx ON table (col_a); AND: CREATE TABLE t ( id INT REFERENCES table_2 (col_b); ); Won't this be confusing and make things hard to remember? Sorry for not asking earlier, I somehow missed this. Regard, Tels -- S

Re: [HACKERS] Parallel Append implementation

2017-03-12 Thread Tels
Moin, On Sat, March 11, 2017 11:29 pm, Robert Haas wrote: > On Fri, Mar 10, 2017 at 6:01 AM, Tels <nospam-pg-ab...@bloodgate.com> > wrote: >> Just a question for me to understand the implementation details vs. the >> strategy: >> >> Have you considered how

Re: [HACKERS] Parallel Append implementation

2017-03-10 Thread Tels
Assign 1 workers to each plan until no workers are left? In the second case you would have 5 plans running in a quasy-sequential manner, which might be slower than the other way. Or not, that probably needs some benchmarks? Likewise, if you have a mix of plans with max workers like: Plan A: 1 wo

Re: [HACKERS] Upgrading postmaster's log messages about bind/listen errors

2017-03-10 Thread Tels
ches. Granted, there should be firewall rules preventing access, but misconfigurations, or simple changes can happen and go unnoticed. If later the postmaster bind address changes, maybe due to an update or human error, you got the stars aligned just right for an unauthorized access. OTOH, that the &q

Re: [HACKERS] PassDownLimitBound for ForeignScan/CustomScan [take-2]

2017-03-10 Thread Tels
lled. Maybe to mirror the comment on "rs_done": /* have we started yet? */ Also, maybe it's easier for the comment to describe what is happening in the code because of the flag, not just to the flag itself: /* To do things once when we are called */ Anyway, it is a minor point and don't let me distract you from your work, I do like the feature and the patch :) Best wishes, Tels -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] Declarative partitioning optimization for large amount of partitions

2017-03-09 Thread Tels
Hi Aleksander, noticed this in your patch: > * Add a corespinding entry to pgStatTabHash. "corresponding" Also a question: Some one-line comments are /* Comment. */ while others are /* * Comment. */ Why the difference? Hope this helps, Tels PS: Sorry if this app

Re: [HACKERS] Declarative partitioning optimization for large amount of partitions

2017-03-09 Thread Tels
Hi Aleksander, noticed this in your patch: > * Add a corespinding entry to pgStatTabHash. "corresponding" Also a question: Some one-line comments are /* Comment. */ while others are /* * Comment. */ Why the difference? Hope this helps, Tels -- Sent via pgsql-hackers

Re: [HACKERS] [PATCH]: fix bug in SP-GiST box_ops

2017-03-09 Thread Tels
ad example for the '&<' case" as test case, too? After all, it should pass the test after the patch. Bets regards, Tels -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers

Re: [HACKERS] PassDownLimitBound for ForeignScan/CustomScan [take-2]

2017-03-01 Thread Tels
these: >+ * Also, pass down the required number of tuples >+ * Pass down the number of required tuples by the upper node And this comment might be better "were we already called?" >+ bool rs_started; /* are we already called? */ Hope