Hi
I am checking new Fedora 26, where new gcc compiler is used.
float.c: In function ‘float4out’:
float.c:382:41: warning: ‘%.*g’ directive output may be truncated writing
between 1 and 310 bytes into a region of size 65 [-Wformat-truncation=]
snprintf(ascii, MAXFLOATWIDTH + 1, "%.*g", ndig,
On Fri, Feb 17, 2017 at 10:35 AM, Rafia Sabih
wrote:
>
>
> On Thu, Feb 16, 2017 at 9:25 PM, Amit Kapila
> wrote:
>>
>>
>> 4.
>> ExecReScanIndexOnlyScan(IndexOnlyScanState *node)
>> {
>> ..
>> + /*
>> + * if we are here to just update the scan keys, then don't reset parallel
>> + * scan
>> + */
>>
On 2/17/17 10:46 PM, Alvaro Herrera wrote:
Jim Nasby wrote:
On 2/17/17 10:19 PM, Alvaro Herrera wrote:
(FWIW, I'm wondering because I was just looking to see why there's no
details for things like altering a column in a table.)
Do you mean you want to have access to the details of the alter ta
Jim Nasby wrote:
> On 2/17/17 10:19 PM, Alvaro Herrera wrote:
> > > (FWIW, I'm wondering because I was just looking to see why there's no
> > > details for things like altering a column in a table.)
> > Do you mean you want to have access to the details of the alter table
> > operations being execu
On 2/17/17 9:53 PM, Alvaro Herrera wrote:
Another way to think about this problem is an approach Peter E suggested
not long ago, which was to change the objname/objargs representation
more completely.
Hrm, I didn't see that. What was the idea?
BTW, I do find it odd (and might eventually find i
On 2/17/17 10:19 PM, Alvaro Herrera wrote:
(FWIW, I'm wondering because I was just looking to see why there's no
details for things like altering a column in a table.)
Do you mean you want to have access to the details of the alter table
operations being executed? There's no structured data for
Jim Nasby wrote:
> I'm confused by this:
>
> "pg_event_trigger_ddl_commands returns one row for each base command
> executed; some commands that are a single SQL sentence may return more than
> one row."
>
> What is a "SQL sentence"?
I meant "a single SQL command". The word "sentence" probably
On 2/14/17 2:39 PM, Andres Freund wrote:
One part of this would need to be having a designated committee of the
Postgres community pick a set of "blessed" extensions for packagers to
package. Right now, contrib serves that purpose (badly). One of the
reasons we haven't dealt with the extension
I'm confused by this:
"pg_event_trigger_ddl_commands returns one row for each base command
executed; some commands that are a single SQL sentence may return more
than one row."
What is a "SQL sentence"?
(FWIW, I'm wondering because I was just looking to see why there's no
details for things
Jim Nasby wrote:
> See below. ISTM that pg_get_object_address should support everything
> pg_identify_object_as_address can output, no?
>
> I'm guessing the answer here is to have pg_identify_object_as_address
> complain if you ask it for something that's not mapable.
Yes, I think we should just
See below. ISTM that pg_get_object_address should support everything
pg_identify_object_as_address can output, no?
I'm guessing the answer here is to have pg_identify_object_as_address
complain if you ask it for something that's not mapable.
~@decina.local/5621# CREATE TYPE comp AS (a int, b
On Fri, Feb 17, 2017 at 9:15 PM, Tom Lane wrote:
> Amit Kapila writes:
>> On Fri, Feb 17, 2017 at 11:22 AM, Tom Lane wrote:
>>> In short, it looks to me like ExecShutdownGatherWorkers doesn't actually
>>> wait for parallel workers to finish (as its comment suggests is
>>> necessary), so that on
On Feb 17, 2017, at 10:31 AM, Magnus Hagander wrote:
>
> On Thu, Feb 16, 2017 at 9:58 PM, David Christensen wrote:
> Extracted from a larger patch, this patch provides the basic infrastructure
> for turning data
> checksums off in a cluster. This also sets up the necessary pg_control
> fields
On Fri, Feb 17, 2017 at 4:22 PM, Tomas Vondra
wrote:
> What about adding a paragraph into pg_basebackup docs, explaining that
> with 'fast' it does immediate checkpoint, while with 'spread' it'll wait
> for a spread checkpoint.
>
I agree that a better, and self-contained, explanation of the beha
On 2/14/17 15:19, Josh Berkus wrote:
> You have to admit that it seems really strange in the eyes of a new user
> that ISN is packaged with PostgreSQL, whereas better-written and more
> popular extensions (like plv8, pg_partman or pgq) are not.
I don't know. Seems pretty standard coming from a Py
On 2/2/17 08:27, Alexey Bashtanov wrote:
> The view information_schema.constraint_column_usage becomes slow when
> the number of columns and constraints raise to substantial values.
> This is because of a join condition that allows only join filter to
> enforce. The patch is to optimize it.
comm
Mark Dilger writes:
>> On Feb 17, 2017, at 3:37 PM, Peter Eisentraut
>> wrote:
>> If your compiler isn't warning about anything with that, then there is
>> something wrong with it.
> $ gcc --version
> Configured with: --prefix=/Applications/Xcode.app/Contents/Developer/usr
> --with-gxx-include
On 2/15/17 10:40, David Christensen wrote:
> Throws a build error if we encounter a different number of fields in a
> DATA() line than we expect for the catalog in question.
>
> Previously, it was possible to silently ignore any mismatches at build
> time which could result in symbol undefined err
On 2/15/17 10:52, Robert Haas wrote:
> Let's wait and see if anybody else has an opinion. I imagine that, as
> further libpq parameters are added, eventually this is going to get
> long and annoying enough that we want to do something about it. But
> we might not have reached that point just yet.
> On Feb 17, 2017, at 3:37 PM, Peter Eisentraut
> wrote:
>
> On 2/17/17 16:13, Mark Dilger wrote:
>> + PGAC_PROG_CC_CFLAGS_OPT([-Wc++-compat])
>
> If your compiler isn't warning about anything with that, then there is
> something wrong with it.
$ gcc --version
Configured with: --prefix=/App
On 2/16/17 09:38, Thom Brown wrote:
> Please find attached a patch to fix 2 typos.
>
> 1) s/mypubclication/mypublication/
>
> 2) Removed trailing comma from last column definition in example.
committed, thanks
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development,
On 2/17/17 16:13, Mark Dilger wrote:
> + PGAC_PROG_CC_CFLAGS_OPT([-Wc++-compat])
If your compiler isn't warning about anything with that, then there is
something wrong with it.
--
Peter Eisentraut http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Trainin
I wrote:
> Right now I'd rather focus on getting to where we can have -Werror on in
> the buildfarm at all. longfin says we've got work to do on that, at least
> in the back branches. It may be that we can't expect near-EOL branches to
> always compile perfectly cleanly on newer compilers.
I fou
Here is a patch set to refine various access control settings in logical
replication. Currently, you need to be replication or superuser for
most things, and the goal of these patches is to allow ordinary users
equipped with explicit privileges to do most things. (Btw., current
documentation is h
On 02/17/2017 08:17 PM, Jim Nasby wrote:
> On 2/14/17 5:18 PM, Robert Haas wrote:
>> On Tue, Feb 14, 2017 at 4:06 PM, Alvaro Herrera
>> wrote:
>>> I'd rather have a --quiet mode instead. If you're running it by hand,
>>> you're likely to omit the switch, whereas when writing the cron job
>>> you
Peter,
* Peter Eisentraut (peter.eisentr...@2ndquadrant.com) wrote:
> On 2/17/17 09:33, Stephen Frost wrote:
> >> One reason is that pg_subscription is only readable by a superuser, so
> >> we can't even dump them unless we're superuser.
> > Sure, but that's true of roles and other things.. On a
On 2/17/17 09:33, Stephen Frost wrote:
>> One reason is that pg_subscription is only readable by a superuser, so
>> we can't even dump them unless we're superuser.
> Sure, but that's true of roles and other things.. On a system with RLS,
> likely only the database superuser can actually read every
Mark Dilger writes:
> if test "$GCC" = yes -a "$ICC" = no; then
>CFLAGS="-Wall -Wmissing-prototypes -Wpointer-arith"
> + PGAC_PROG_CC_CFLAGS_OPT([-Wempty-body])
> + PGAC_PROG_CC_CFLAGS_OPT([-Wignored-qualifiers])
> + PGAC_PROG_CC_CFLAGS_OPT([-Wimplicit-fallthrough])
> + PGAC_PROG_CC_CFLAG
On February 17, 2017 1:13:10 PM PST, Mark Dilger
wrote:
>How about we add (some of) these extra warnings, plus -Werror,
>in a section that is only active for platforms/compilers where we
>know there aren't spurious warnings? That would make detecting
>unintentionally introduced warnings simple
> On Feb 17, 2017, at 12:21 PM, Tom Lane wrote:
>
> Thomas Munro writes:
>> On Sat, Feb 18, 2017 at 9:04 AM, Tom Lane wrote:
>>> [ pokes around... ] Ah, that's called COPT, and it's entirely
>>> undocumented :-(. Probably ought to fix that.
>
>> One way to set that up is like this:
>
>> $
On 2/14/17 5:18 PM, Robert Haas wrote:
On Tue, Feb 14, 2017 at 4:06 PM, Alvaro Herrera
wrote:
I'd rather have a --quiet mode instead. If you're running it by hand,
you're likely to omit the switch, whereas when writing the cron job
you're going to notice lack of switch even before you let the
Thomas Munro writes:
> On Sat, Feb 18, 2017 at 9:04 AM, Tom Lane wrote:
>> [ pokes around... ] Ah, that's called COPT, and it's entirely
>> undocumented :-(. Probably ought to fix that.
> One way to set that up is like this:
> $ cat src/Makefile.custom
> COPT=-Wall -Werror $(CC_OPT)
Well, we
On Sat, Feb 18, 2017 at 9:04 AM, Tom Lane wrote:
> Andres Freund writes:
>> On February 17, 2017 11:40:57 AM PST, Tom Lane wrote:
>>> AFAICS, if you want to build with -Werror, you have to configure
>>> without that and then inject it afterwards. I wonder how people who
>>> use -Werror are hand
Andres Freund writes:
> On February 17, 2017 11:40:57 AM PST, Tom Lane wrote:
>> AFAICS, if you want to build with -Werror, you have to configure
>> without that and then inject it afterwards. I wonder how people who
>> use -Werror are handling that. Is it possible to do at all in a
>> buildfar
> On Feb 17, 2017, at 11:40 AM, Tom Lane wrote:
>
> I wrote:
>> Yeah. I have longfin which is running Apple's clang, and is on a machine
>> that doesn't have much to do otherwise. I propose to turn on -Werror in
>> its configuration, and to configure a second critter on the same hardware
>> th
On February 17, 2017 11:40:57 AM PST, Tom Lane wrote:
>I wrote:
>> Yeah. I have longfin which is running Apple's clang, and is on a
>machine
>> that doesn't have much to do otherwise. I propose to turn on -Werror
>in
>> its configuration, and to configure a second critter on the same
>hardware
I wrote:
> Yeah. I have longfin which is running Apple's clang, and is on a machine
> that doesn't have much to do otherwise. I propose to turn on -Werror in
> its configuration, and to configure a second critter on the same hardware
> that runs with -Werror as well as --disable-integer-datetimes
On 2/15/17 1:37 PM, Ryan Murphy wrote:
attcacheoff can only be set positive for fields preceding any varlena
(typlen<0, but including the first such) or nullable values. I don't
know how much faster it is with the cache; you can measure it if your
curiosity is strong enough -- j
On 2/15/17 12:05 PM, Alvaro Herrera wrote:
Jim Nasby wrote:
ISTR previous discussion of allowing more stats files; if that happened
I think having stats that were dedicated to (auto)vacuum would be very
useful. That's clearly a lot more work though.
What?
There's a bunch of information repor
On 2/14/17 2:49 PM, Pavel Stehule wrote:
> Tom's use case might be more easily served by specifying a
> template database. I don't think Pavel ever posted his use case.
Wait, that's precisely what Pavel asked?
I would to use regress test environment in my current case. 99% code in
Alvaro Herrera writes:
> Tom Lane wrote:
>> My own compilers don't generate errors either, only warnings. Maybe
>> we could catch this sort of thing mechanically with a critter configured
>> with -Werror as well as --disable-integer-datetimes, but I'm a tad
>> hesitant to have a buildfarm machine
Thomas Munro pointed out that commit 7c030783a broke things on
--disable-integer-datetimes builds, because somebody cleverly used
TimestampTz to declare timestamp variables, no doubt not having
read the comment (which doesn't even appear in the same file :-()
that
* ... The replication protocol a
Tom Lane wrote:
> My own compilers don't generate errors either, only warnings. Maybe
> we could catch this sort of thing mechanically with a critter configured
> with -Werror as well as --disable-integer-datetimes, but I'm a tad
> hesitant to have a buildfarm machine configured with -Werror. We
On Fri, Feb 17, 2017 at 9:31 AM, Tom Lane wrote:
> This seems like it'd be quite a different tool than amcheck, though.
> Also, it would only find broken-HOT-chain corruption, which might be
> a rare enough issue to not deserve a single-purpose tool.
FWIW, my ambition for amcheck is that it will
> On Feb 17, 2017, at 10:31 AM, Magnus Hagander wrote:
>
> Per the point made by somebody else (I think Simon?) on the other thread, I
> think it also needs WAL support. Otherwise you turn it off on the master, but
> it remains on on a replica which will cause failures once datablocks without
I wrote:
> However, you might be able to find it without so much random I/O.
> I'm envisioning a seqscan over the table, in which you simply look for
> HOT chains in which the indexed columns aren't all the same. When you
> find one, you'd have to do a pretty expensive index lookup to confirm
> wh
On Feb 17, 2017, at 12:54 AM, Magnus Hagander wrote:
> If we could somehow integrate PGXN with both the RPM build process, the DEB
> build process and a Windows build process (whether driven by PGXN or just
> "fed enough data" by PGXN is a different question), I think that would go a
> long wa
On 17 February 2017 at 14:14, Peter Eisentraut
wrote:
> On 2/16/17 09:44, Thom Brown wrote:
>> I've noticed that when creating a subscription, it can't be
>> interrupted. One must wait until it times out, which takes just over
>> 2 minutes. I'm guessing ALTER SUBSCRIPTION would have the same
>>
Peter Geoghegan writes:
> The difference with a test that could detect this variety of
> corruption is that that would need to visit the heap, which tends to
> be much larger than any one index, or even all indexes. That would
> probably need to be random I/O, too. It might be possible to mostly
>
I wrote:
> Thomas Munro writes:
>> Since commit 7c030783a5bd07cadffc2a1018bc33119a4c7505 it seems that $SUBJECT.
> Hmm ... I thought we had at least one buildfarm member still testing
> --disable-integer-datetimes. Evidently somebody needs to close that
> gap.
The plot thickens: we *do* have on
On Fri, Feb 17, 2017 at 8:23 AM, Keith Fiske wrote:
> It's not the load I'm worried about, it's the locks that are required at
> some point during the rebuild. Doing an index rebuild here and there isn't a
> big deal, but trying to do it for an entire heavily loaded, multi-terabyte
> database is h
Robert Haas writes:
> I'm thinking we should change this to look more like the
> MemoryContextAlloc interface. Let's have DSA_ALLOC_HUGE,
> DSA_ALLOC_NO_OOM, and DSA_ALLOC_ZERO, just like the corresponding
> MCXT_* flags, and a function dsa_allocate_extended() that takes a
> flags argument. Then
Keith Fiske wrote:
> I can understandable if it's simply not possible, but if it is, I think in
> any cases of data corruption, having some means to check for it to be sure
> you're in the clear would be useful.
Maybe it is possible. I just didn't try, since it didn't seem very
useful.
--
Álva
On Thu, Feb 16, 2017 at 9:58 PM, David Christensen
wrote:
> Extracted from a larger patch, this patch provides the basic
> infrastructure for turning data
> checksums off in a cluster. This also sets up the necessary pg_control
> fields to support the
> necessary multiple states for handling the
On 16 February 2017 at 20:37, Robert Haas wrote:
> I'm not sure that it's going to be useful to make this logic very
> complicated. I think the most important thing is to give 1 worker to
> each plan before we give a second worker to any plan. In general I
> think it's sufficient to assign a wo
Ashutosh Bapat wrote:
> Do we have any performance measurements where we see that Goal B
> performs better than Goal A, in such a situation? Do we have any
> performance measurement comparing these two approaches in other
> situations. If implementation for Goal B beats that of Goal A always,
> we
On Fri, Feb 17, 2017 at 11:12 AM, Alvaro Herrera
wrote:
> Keith Fiske wrote:
>
> > Was just curious if anyone was able to come up with any sort of method to
> > test whether an index was corrupted by this bug, other than just waiting
> > for bad query results? We've used concurrent index rebuildi
Stephen Frost writes:
> * Alvaro Herrera (alvhe...@2ndquadrant.com) wrote:
>> Magnus Hagander wrote:
>>> I'm guessing if we backpatch something like that, it would cause issues for
>>> translations, right? So we should make it head only?
>> We've had the argument a number of times. My stand is t
Thomas Munro writes:
> Since commit 7c030783a5bd07cadffc2a1018bc33119a4c7505 it seems that $SUBJECT.
Hmm ... I thought we had at least one buildfarm member still testing
--disable-integer-datetimes. Evidently somebody needs to close that
gap.
> Maybe the attached is the right fix for that?
I'l
Keith Fiske wrote:
> Was just curious if anyone was able to come up with any sort of method to
> test whether an index was corrupted by this bug, other than just waiting
> for bad query results? We've used concurrent index rebuilding quite
> extensively over the years to remove bloat from busy sys
Robert Haas writes:
> On Fri, Feb 17, 2017 at 6:51 AM, Greg Stark wrote:
>> Moreover, it wouldn't be hard to make sum(float4) use a float8 as an
>> accumulator and then cast to float4 for the final state. That would be
>> 100% compatible with the existing behaviour aside from producing more
>> ac
Amit Kapila writes:
> On Fri, Feb 17, 2017 at 11:22 AM, Tom Lane wrote:
>> In short, it looks to me like ExecShutdownGatherWorkers doesn't actually
>> wait for parallel workers to finish (as its comment suggests is
>> necessary), so that on not-too-speedy machines the worker slots may all
>> stil
On Mon, Feb 6, 2017 at 10:17 PM, Amit Kapila
wrote:
> On Mon, Feb 6, 2017 at 10:28 PM, Tom Lane wrote:
> > Amit Kapila writes:
> >> Hmm. Consider that the first time relcahe invalidation occurs while
> >> computing id_attrs, so now the retry logic will compute the correct
> >> set of attrs (co
Thomas Munro wrote:
> Hi,
>
> Since commit 7c030783a5bd07cadffc2a1018bc33119a4c7505 it seems that $SUBJECT.
Added it to open items list.
--
Álvaro Herrerahttps://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
--
Sent via pgsql-hac
On 2/15/17 17:55, Petr Jelinek wrote:
> I am not quite convinced that this should be handled by logical decoding
> itself. It's quite possible to have output plugins that will handle this
> correctly for their use-cases (by doing similar conversion you did in
> the original patch) so they should no
* Alvaro Herrera (alvhe...@2ndquadrant.com) wrote:
> Magnus Hagander wrote:
> > On Wednesday, February 15, 2017, Alvaro Herrera
> > wrote:
> >
> > > Magnus Hagander wrote:
> > >
> > > > printf(_(" -R, --write-recovery-conf\n"
> > > > - " write r
Hi,
When working on REINDEX CONCURRENTLY I noticed that the new memory
context created in the ReindexMultipleTables() seems pointless.
The purpose claimed in the code for introducing the
ReindexMultipleTables context is to make sure the list we build with
relation IDs survive the commit, sin
* Peter Eisentraut (peter.eisentr...@2ndquadrant.com) wrote:
> On 2/13/17 12:07, Fujii Masao wrote:
> > Anyway IMO that we can expose all the
> > columns except the sensitive information (i.e., subconninfo field)
> > in pg_subscription to even non-superusers.
>
> You mean with column privileges?
>
Peter,
* Peter Eisentraut (peter.eisentr...@2ndquadrant.com) wrote:
> On 2/16/17 21:04, Stephen Frost wrote:
> > I'm not entirely sure about the reasoning behind requiring a flag to
> > include subscriptions in pg_dump output,
>
> One reason is that pg_subscription is only readable by a superuser
On 2/13/17 12:07, Fujii Masao wrote:
> Anyway IMO that we can expose all the
> columns except the sensitive information (i.e., subconninfo field)
> in pg_subscription to even non-superusers.
You mean with column privileges?
We could probably do that. I don't know if we have done that before on
s
On 2/16/17 09:44, Thom Brown wrote:
> I've noticed that when creating a subscription, it can't be
> interrupted. One must wait until it times out, which takes just over
> 2 minutes. I'm guessing ALTER SUBSCRIPTION would have the same
> problem.
>
> Shouldn't we have an interrupt for this?
I thi
On 2/16/17 21:04, Stephen Frost wrote:
> I'm not entirely sure about the reasoning behind requiring a flag to
> include subscriptions in pg_dump output,
One reason is that pg_subscription is only readable by a superuser, so
we can't even dump them unless we're superuser.
Also, restoring a subscri
On Fri, Feb 17, 2017 at 10:43 PM, Andreas Karlsson wrote:
> Thinking about this makes me wonder about why you decided to use a
> transaction per index in many of the steps rather than a transaction per
> step. Most steps should be quick. The only steps I think the makes sense to
> have a transacti
On Wed, Feb 15, 2017 at 9:27 PM, Michael Paquier
wrote:
> On Wed, Feb 15, 2017 at 7:58 PM, Heikki Linnakangas wrote:
>> On 02/09/2017 09:33 AM, Michael Paquier wrote:
>>> Now regarding the shape of the implementation for SCRAM, we need one
>>> thing: a set of routines in src/common/ to build deco
Hi all,
As of now, we expand the hash index by doubling the number of bucket
blocks. But unfortunately, those blocks will not be used immediately.
So I thought if we can differ bucket block allocation by some factor,
hash index size can grow much efficiently. I have written a POC patch
which does
On 02/17/2017 01:53 PM, Andreas Karlsson wrote:
I am actually thinking about going the opposite direction (by reducing
the number of times we call WaitForLockers), because it is not just
about consuming transaction IDs, we also do not want to wait too many
times for transactions to commit. I am l
Magnus Hagander wrote:
> On Wednesday, February 15, 2017, Alvaro Herrera
> wrote:
>
> > Magnus Hagander wrote:
> >
> > > printf(_(" -R, --write-recovery-conf\n"
> > > - " write recovery.conf
> > after backup\n"));
> > > + "
Amit,
* Amit Langote (langote_amit...@lab.ntt.co.jp) wrote:
> In certain cases, pg_dump's dumpTableSchema() emits a separate ALTER TABLE
> command for those schema elements of a table that could not be included
> directly in the CREATE TABLE command for the table.
Any chance we could start adding
On Fri, Feb 17, 2017 at 6:51 AM, Greg Stark wrote:
> On 15 February 2017 at 12:52, Robert Haas wrote:
>> Personally, I find it somewhere in the middle: I think the way it
>> works now is reasonable, and I think what he wants would have been
>> reasonable as well. However, I find it hard to belie
On Thu, Feb 16, 2017 at 10:09 PM, Michael Paquier
wrote:
> On Fri, Feb 17, 2017 at 12:03 PM, Thomas Munro
> wrote:
>> On Fri, Feb 17, 2017 at 11:34 AM, Thomas Munro
>> wrote:
>>> On Fri, Feb 17, 2017 at 7:02 AM, Robert Haas wrote:
http://git.postgresql.org/pg/commitdiff/9acb85597f1223ac26a
On Fri, Feb 17, 2017 at 4:47 PM, Amit Kapila
wrote:
> On Fri, Feb 17, 2017 at 3:59 PM, Thomas Munro
> wrote:
> > On Thu, Feb 2, 2017 at 2:32 AM, Rushabh Lathia
> wrote:
> >> Please find attached latest patch.
> >
> > The latest patch still applies (with some fuzz), builds and the
> > regression
On 02/14/2017 04:56 AM, Michael Paquier wrote:
On Tue, Feb 14, 2017 at 11:32 AM, Andreas Karlsson wrote:
On 02/13/2017 06:31 AM, Michael Paquier wrote:
Er, something like that as well, no?
DETAIL: CPU: user: 0.00 s, system: 0.00 s, elapsed: 0.00 s.
REINDEX (VERBOSE) currently prints one suc
On Fri, Feb 17, 2017 at 11:22 AM, Tom Lane wrote:
> Buildfarm members gaur, pademelon, and gharial have all recently shown
> failures like this:
>
> ***
> /home/bfarm/bf-data/HEAD/pgsql.build/src/test/regress/expected/select_parallel.out
> Thu Feb 16 20:35:14 2017
> ---
> /home/bfarm/bf-data/H
On Fri, Feb 17, 2017 at 3:59 PM, Thomas Munro wrote:
> On Thu, Feb 2, 2017 at 2:32 AM, Rushabh Lathia
> wrote:
> > Please find attached latest patch.
>
> The latest patch still applies (with some fuzz), builds and the
> regression tests pass.
>
> Attached latest patch, which applies cleanly on l
On 15 February 2017 at 12:52, Robert Haas wrote:
> Personally, I find it somewhere in the middle: I think the way it
> works now is reasonable, and I think what he wants would have been
> reasonable as well. However, I find it hard to believe it would be
> worth changing now on backward compatibi
On Fri, Feb 17, 2017 at 3:59 PM, Thomas Munro
wrote:
> On Thu, Feb 2, 2017 at 2:32 AM, Rushabh Lathia
> wrote:
>> Please find attached latest patch.
>
> The latest patch still applies (with some fuzz), builds and the
> regression tests pass.
>
> I see that Robert made a number of changes and pos
On Thu, Feb 2, 2017 at 2:32 AM, Rushabh Lathia wrote:
> Please find attached latest patch.
The latest patch still applies (with some fuzz), builds and the
regression tests pass.
I see that Robert made a number of changes and posted a v6 along with
some numbers which he described as lacklustre, b
On Fri, Feb 17, 2017 at 12:44 PM, Ashutosh Sharma
wrote:
> >> On Thu, Feb 16, 2017 at 5:07 AM, Alexander Korotkov
> >> wrote:
> >> > On Wed, Feb 15, 2017 at 8:49 PM, Alvaro Herrera
> >> >
> >> > wrote:
> >> >> Alexander Korotkov wrote:
> >> >>
> >> >> > Difference between master, pgxact-align-2
Hi,
>> On Thu, Feb 16, 2017 at 5:07 AM, Alexander Korotkov
>> wrote:
>> > On Wed, Feb 15, 2017 at 8:49 PM, Alvaro Herrera
>> >
>> > wrote:
>> >> Alexander Korotkov wrote:
>> >>
>> >> > Difference between master, pgxact-align-2 and pgxact-align-3 doesn't
>> >> > exceed
>> >> > per run variation.
On 16 February 2017 at 20:37, Robert Haas wrote:
> On Thu, Feb 16, 2017 at 1:34 AM, Amit Khandekar
> wrote:
>>> What I was thinking about is something like this:
>>>
>>> 1. First, take the maximum parallel_workers value from among all the
>>> children.
>>>
>>> 2. Second, compute log2(num_childr
On Thu, Feb 16, 2017 at 4:58 PM, Robert Haas wrote:
> On Thu, Feb 16, 2017 at 5:07 AM, Alexander Korotkov
> wrote:
> > On Wed, Feb 15, 2017 at 8:49 PM, Alvaro Herrera <
> alvhe...@2ndquadrant.com>
> > wrote:
> >> Alexander Korotkov wrote:
> >>
> >> > Difference between master, pgxact-align-2 and
On Wednesday, February 15, 2017, Alvaro Herrera
wrote:
> Magnus Hagander wrote:
>
> > printf(_(" -R, --write-recovery-conf\n"
> > - " write recovery.conf
> after backup\n"));
> > + " write recovery.co
On Tuesday, February 14, 2017, David E. Wheeler
wrote:
> On Feb 14, 2017, at 9:37 AM, Magnus Hagander > wrote:
>
> > It's a failing in one of the two at least. It either needs to be easier
> to build the things on windows, or pgxn would need to learn to do binary
> distributions.
>
> PGXN makes
In certain cases, pg_dump's dumpTableSchema() emits a separate ALTER TABLE
command for those schema elements of a table that could not be included
directly in the CREATE TABLE command for the table.
For example:
create table p (a int, b int) partition by range (a);
create table p1 partition of p
94 matches
Mail list logo