The following review has been posted through the commitfest application:
make installcheck-world: tested, failed
Implements feature: tested, passed
Spec compliant: tested, passed
Documentation:tested, passed
Hello Anthony,
Great job!
I decided to take a closer look o
t. Such behavior is
obviously a bug. So as an alternative we could just return an error in
response to ALTER TABLE ... ENABLE REPLICA TRIGGER query for triggers
that we know will never be executed.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
.. ok
```
After:
```
t/001_rep_changes.pl .. ok
t/002_types.pl ok
t/003_constraints.pl .. ok
t/004_sync.pl . ok
t/005_encoding.pl . ok
t/006_rewrite.pl .. ok
t/007_ddl.pl .. ok
All tests successful.
```
--
Best regards,
Aleksander Alekseev
diff --g
t;test_pkey"
[3227] DETAIL: Key (k)=(k1) already exists.
[3176] LOG: worker process: logical replication worker for subscription 16402
(PID 3227) exited with exit code 1
```
What do you think?
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
egarding trying different
architectures, e.g. x64, x86, arm.
[1]: https://www.postgresql.org/message-id/20160321130850.6ed6f598%40fujitsu
[2]: https://github.com/afiskon/pgscripts/blob/master/valgrind.sh
[3]: https://github.com/afiskon/pgscripts/blob/master/static-analysis.sh
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
f this patch, and the page doesn't say so.
> Something wrong with your script?
It's a bug. The authors were determined by the senders of the first
email in the corresponding thread. This is because I needed email list
to CC the authors but the commitfest application doesn't show emails
to "Needs
Review". On the bright side while doing this I've noticed that many
patches were already updated by their authors.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
ou submitted it.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: tested, passed
Documentation:tested, passed
Also I didn't manage to find any typos or obvious mistakes in
Hi Andreas,
On 09/12/2017 04:14 PM, Aleksander Alekseev wrote:
> > Title: Foreign Key Arrays
> > Author: Mark Rofail
> > URL:https://commitfest.postgresql.org/14/1252/
>
> I am currently reviewing this one and it applies, compiles, and passes the
> test suite.
types
Author: Anton Dignös
URL: https://commitfest.postgresql.org/14/839/
Needs Review Total: 120
Failed Total: 32 (26.67 %)
As always, any feedback is very welcomed!
[1]: http://commitfest.cputube.org/
[2]:
https://postgr.es/m/CAB7nPqSrHF%2BkNQ6Eq2uy91RcysoCzQr1AjOjkuCn9jvMdJZ6Fw%40mail.gm
is I'll re-run the script to
make sure that the list is up-to-date). I'm also going to write one more
email with CC to the authors of these patches to let them know that the
status of their patch has changed.
[1]: https://github.com/afiskon/py-autoreviewer
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
tfest
application as is and provide corresponding checkboxes in the web
interface of the "autoreviewer" application. Naturally every
automatically generated code review will include a link that disables
autotests for this particular commitfest entry.
I hope this observation will chan
y objections to give this idea a try I'm willing to
write and host a corresponding script.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
When and
if it will happen nothing prevents us from adding this code back. So
it's +1.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: tested, passed
Documentation:tested, passed
LGTM.
The new status of this patch is: Ready for Committer
The following review has been posted through the commitfest application:
make installcheck-world: tested, failed
Implements feature: not tested
Spec compliant: not tested
Documentation:not tested
PostgreSQL fails with SIGSEGV during `make check-world`.
Backtrace: http
The following review has been posted through the commitfest application:
make installcheck-world: not tested
Implements feature: not tested
Spec compliant: not tested
Documentation:not tested
Hi Emre,
I'm afraid these patches conflict with current master branch.
The
The following review has been posted through the commitfest application:
make installcheck-world: not tested
Implements feature: not tested
Spec compliant: not tested
Documentation:not tested
Hi Craig,
I'm afraid patches 0002 and 0003 don't apply anymore. Could you pl
The following review has been posted through the commitfest application:
make installcheck-world: not tested
Implements feature: not tested
Spec compliant: not tested
Documentation:not tested
Hi Marina,
I'm sorry to inform you that the v5 path set become a little outd
nsion export_to_gnuplot;
select * from data
\export_to_gnuplot | gnuplot ...
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
h about the required or recommended length of the
salt.
I personally believe that 2^96 of possible salts is consistent with both
RFCs and should be enough in practice.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
e an
> issue.
>
> [1]: https://bugs.archlinux.org/task/55134
TWIMC I've also described a workaround there.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
ssue.
[1]: https://bugs.archlinux.org/task/55134
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
hard drive.
[1]:
http://www.pgcon.org/2017/schedule/attachments/453_corruption-pgcon-2017.pdf
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: tested, passed
Documentation:tested, passed
I believe this patch is "Ready for Committer".
The new statu
Hi Victor,
> I forgot to attach the patch. Sorry.
> Here it is.
I would say that this patch is in a pretty good shape now. And I see a
99% code coverage of rbtree.c. Let's see what committers think.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
your patch to the nearest commitfest [1].
Otherwise there is a good chance that everyone will forget about it
quite soon.
[1]: https://commitfest.postgresql.org/14/
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
and
be sure that it actually works when someone will need the rest of its
functionality?
[1]: https://github.com/afiskon/pgscripts/blob/master/code-coverage.sh
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
t. For similar reason I would advise to add this patch to the next
commitfest.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
of
> the logical replication?
I knew about TRUNCATE and it most definitely should be documented if
it's not yet. And what about large objects?
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
is misguided.
In the same time implementing a plugable storage API and then implementing
encrypted / compressed / whatever storage in a standalone extension using
this API seems to be a reasonable thing to do.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
den to ask a colleague for
assistance in case you don't have sufficient permissions to do
something. And I got a strong feeling that solving bureaucracy issues of
specific organizations by changing PostgreSQL core in very invasive way
(keeping in mind testing, maintaining, etc) is misguided.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
few companies that hired system administrators that are too
lazy to read two or three man pages is not a reason to re-implement file
system encryption (or compression, or mirroring if that matters) in any
open source RDBMS.
[1] http://eax.me/dm-crypt/
[2] http://eax.me/freebsd-geli/
--
Bes
Hi Marina,
I still don't see anything particularly wrong with your patch. It
applies, passes all test, it is well test-covered and even documented.
Also I've run `make installcheck` under Valgrind and didn't find any
memory-related errors.
Is there anything that you would like to change before we
//docs.basho.com/riak/kv/2.2.3/learn/concepts/active-anti-entropy/
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
> 83000tps and 86000tps is not so obvious in NUMA system).
I don't see any patch in my email client or pgsql-hackers@ archive. I
would also recommend to add your patch to the nearest commitfest [1].
[1] https://commitfest.postgresql.org/
--
Best regards,
Aleksander Alekseev
signatu
there is no obvious performance degradation on regular
queries (according to pgbench).
In short, it looks very promising.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
Hi Heikki,
> psql: SCRAM authentication requires libpq version 10 or above
Sounds good.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
r
messages using regular expressions or something like this. But I don't
think it's something we should really bother about.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
Hi Heikki,
> psql: SCRAM authentication not supported by this version of libpq
Maybe it would be better to specify a minimum required version?
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
WriteMethod could
probably be improved. Currently there is none.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
ctions
regarding this idea I'm willing to write such a patch.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
Hi Yorick,
> should do so on any standby servers first, before applying the changes to
What you actually meant probably was "do so on ALL standby servers
first", right?
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
Hi Robert,
> Thanks. Please add this to the next CommitFest, as there seems to be
> no urgency (and some risk) in committing it right before feature
> freeze.
Sure. Already done [1].
[1] https://commitfest.postgresql.org/14/1097/
--
Best regards,
Aleksander Alekseev
sign
s said.
I see no problem here. After all, it's just warnings. We can even add
a GUC that disables them specially for experienced users who knows what
she or he is doing. And/or add special case for postgresql.auto.conf.
--
Best regards,
Aleksander Alekseev
pgphtrO1vN7g6.pgp
Description: OpenPGP digital signature
c.
I suggest to warn users about duplicated parameters. Here is a
corresponding patch.
Thoughts?
--
Best regards,
Aleksander Alekseev
diff --git a/src/backend/utils/misc/guc-file.l b/src/backend/utils/misc/guc-file.l
index f01b814..6aa60a4 100644
--- a/src/backend/utils/misc/guc-file.l
+++ b/src/ba
egards,
Aleksander Alekseev
pgpRyayBlz4tm.pgp
Description: OpenPGP digital signature
r and nothing wrong happened.
--
Best regards,
Aleksander Alekseev
pgpqfeUI2TpqL.pgp
Description: OpenPGP digital signature
is patch should go
> a little further and do all of that together.
Thank you for sharing your thoughts on this patch. Here is a new
version.
--
Best regards,
Aleksander Alekseev
diff --git a/src/backend/access/nbtree/nbtinsert.c b/src/backend/access/nbtree/nbtinsert.c
index 6dca8109fd..4d
Hi,
Turned out that there is an unused argument `isroot` in
`btree_xlog_split` procedure. Suggested patch fixes it.
This issue was discovered by Anastasia Lubennikova, coding is done by me.
--
Best regards,
Aleksander Alekseev
diff --git a/src/backend/access/nbtree/nbtxlog.c b/src/backend
good thing.
What about all these memory leak reports [1]? If I see them should I just
ignore them or, if reports look false positive, suggest a patch that
modifies a Valgrind suppression file? In other words what is current
consensus in community regarding Valgrind and it's reports?
[1] h
.sh
[3] http://afiskon.ru/s/88/93c544e6cf_test.sql
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
; Any objection from reviewers to push both patches?
These patches look OK. Definitely no objections from me.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
(e.g.
> join_rel_list + join_rel_hash in PlannerInfo, ResourceArray in
> resowner.c, etc).
I've checked it [1]. Apparently in this particular case patches make code
faster on small amount of partitions as well.
[1] https://postgr.es/m/20170324132258.GB16830%40e733.localdomain
--
gree, fixed.
> > step1 Again, pgstat_report_stat(), all-zero entries aren't deleted from hash
> > although they will be free from point of view of pgStatTabList.
Good point! Fixed.
--
Best regards,
Aleksander Alekseev
diff --git a/src/backend/postmaster/pgstat.c b/src/backend/po
his
matter. Unless there is anyone else who is already working on this
optimization we would like to work on it together. Unfortunately there
is no patch or design discussion of partition-pruning on this
commitfest.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
x27;t make things worse in any regard.
Perf shows that both bottlenecks are gone. Before [1] and after [2].
[1] http://afiskon.ru/s/00/2008c4ae66_temp.png
[2] http://afiskon.ru/s/a5/fd81628a3a_temp.png
On Fri, Mar 24, 2017 at 03:17:44PM +0300, Aleksander Alekseev wrote:
> Hi Anastasia,
>
>
ght to do the trick); whereas I do not think
> we'd remember to de-kluge port.h if we kluge around it there.
>
> I'm fine with leaving it alone, too.
>
> regards, tom lane
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
The following review has been posted through the commitfest application:
make installcheck-world: tested, passed
Implements feature: tested, passed
Spec compliant: tested, passed
Documentation:tested, passed
This patch looks good to me. As I understand we have both a c
)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
--
Best regards,
Aleksander Alekseev
diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c
index 7cacb1e9b2..a22a5a37c8 100644
--- a/src/backend/postmaster/pgstat.c
+++ b/src/bac
d?
[1] https://wiki.postgresql.org/wiki/Valgrind
[2] http://afiskon.ru/s/8a/390698e914_valgrind.tgz
[3] http://afiskon.ru/s/09/c4f6231679_pgvg.txt
[4] https://github.com/afiskon/pgscripts/blob/master/quick-build.sh
[5] https://github.com/afiskon/pgscripts/blob/master/valgrind.sh
--
Best regards,
Aleksan
xactly what I would call a solution. For instance on FreeBSD
Clang is a default compiler and many users build a software from source
code (FreeBSD ports). For some reason I doubt that many of them know
about these flags.
On Wed, Mar 15, 2017 at 02:25:38AM +, Noah Misch wrote:
> On Mon, Mar
needed and the value compatibility is checked by the compiler? I
> > guess there would be some more changes (question is how much), but it
> > would be cleaner.
>
> There's been no discussion or new patch on this thread recently. If you
> are planning to address the issue
> 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 di
could be done here as well then.
On Tue, Mar 07, 2017 at 10:55:12AM +0900, Amit Langote wrote:
> Hi Aleksander,
>
> On 2017/03/07 0:22, Aleksander Alekseev wrote:
> > Hello.
> >
> > OK, here is a patch.
> >
> > Benchmark, before:
> >
> > ``
tion.
Agree, fixed.
On Mon, Mar 06, 2017 at 12:01:50PM -0800, Andres Freund wrote:
> Hi,
>
> This issue has bothered me in non-partitioned use-cases recently, so
> thanks for taking it up.
>
>
> On 2017-03-06 18:22:17 +0300, Aleksander Alekseev wrote:
> > diff --g
sure that this code is
test covered. An exact script I'm using could be found here [1].
[1] https://github.com/afiskon/pgscripts/blob/master/code-coverage.sh
On Wed, Mar 01, 2017 at 10:36:24AM +0900, Amit Langote wrote:
> Hi,
>
> On 2017/02/28 23:25, Aleksander Alekseev wrote:
&g
making the DETACH/ATTACH manipulation, though. But IMO it's not flexible
> case.
>
> It would be a good thing if a regular table could be partitioned through
> separate command. Then your idea would not be restrictive.
>
>
> --
> Maksim Milyutin
> Postgr
ion.diffs.txt
[2] http://afiskon.ru/s/76/a4fb71739c_full-build.sh.txt
On Wed, Mar 01, 2017 at 06:10:10PM +0900, Yugo Nagata wrote:
> Hi Aleksander,
>
> On Tue, 28 Feb 2017 18:05:36 +0300
> Aleksander Alekseev wrote:
>
> > Hi, Yugo.
> >
> > Looks like a great feature! I&#x
/* list of expressions in the partition
> key;
>* one item for
> each zero entry in partattrs[] */
> #endif
> +
> +
> } FormData_pg_partitioned_table;
>
> /* ----
> @@ -62,13 +67,15 @
1] http://afiskon.ru/s/e3/5f47af9102_benchmark.txt
[2] http://afiskon.ru/s/00/2008c4ae66_temp.png
[3] http://afiskon.ru/s/23/650f0afc89_stack.txt
[4] http://afiskon.ru/s/03/a7e685a4db_stack2.txt
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
In theory - could we just always use our internal strl* implementations?
On Mon, Feb 20, 2017 at 09:26:44AM -0500, Tom Lane wrote:
> Aleksander Alekseev writes:
> > I've just tried to build PostgreSQL with Clang 3.9.1 (default version
> > currently available in Arch Linux)
we could easily deal with:
```
xloginsert.c:742:18: warning: implicit conversion from 'int' to 'char'
changes value from 253 to -3 [-Wconstant-conversion]
```
Patch that fixes these warnings is attached to this email.
[1] http://lists.llvm.org/pipermail/cfe-dev/2016-March/048126.html
--
B
and strdup calls)
* scram_Normalize doesn't check malloc return value
Sorry for lots of emails.
On Mon, Feb 20, 2017 at 03:15:14PM +0300, Aleksander Alekseev wrote:
> Speaking about flaws, it looks like there is a memory leak in
> array_to_utf procedure - result is allocated twice.
>
Speaking about flaws, it looks like there is a memory leak in
array_to_utf procedure - result is allocated twice.
On Mon, Feb 20, 2017 at 02:51:13PM +0300, Aleksander Alekseev wrote:
> Hi!
>
> Currently I don't see any significant flaws in these patches. However I
> would li
I have is: do we really want to have
> password_encryption being set to "scram" for verifiers of
> SCRAM-SHA-256? I would think that scram_sha_256 makes the most sense.
> Who knows, perhaps there could be in a couple of years a SHA-SHA-512..
>
> At the same time, attach
No, I'm afraid `make distclean` doesn't help. I've re-checked twice.
On Mon, Feb 06, 2017 at 12:52:11PM -0300, Alvaro Herrera wrote:
> Aleksander Alekseev wrote:
> > Hello.
> >
> > Good to know that the work on this great feature continues!
> >
>
to leave that for a follow-up commit, after these other patches have been
> committed, so we can discuss that work separately.
>
> These are also available on Michael's github repository, at
> https://github.com/michaelpq/postgres/tree/scram.
>
> - Heikki
>
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
s is always
> > 0 and the corresponding assertion fails in the next test:
> >
> > [patched]=# select json_to_record('{"a": [1, 2, 3]}') as rec(a int[]);
> >
> >
> > Should I submit these patches to commitfest?
>
> It seems to me that it would be a good idea to add them.
> --
> Michael
>
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
12:03, Haribabu Kommi
> wrote:
>
> > > Moved to next CF with "needs review" status.
>
> Looks like we stuck here little bit. Does anyone else have any
> suggestions/improvements, or this patch is in good enough shape?
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
alculate current year from number 534873794643886 like this:
```
>>> int(2000 + 534873794643886 / 1000 / 1000 / 60 / 60 / 24 / 365.2425)
2016
```
I suggest to rewrite the documentation a bit to make it more clear that
by default timestamp is stored in microseconds. Corresponding patch
Whether it's worth worrying about, I dunno. This is hardly the only
> C idiom you need to be familiar with to read the PG code.
Well, at least this v2 version of the patch removes second string
scanning. And I still believe that this inlined strspn is not readable
or obvious at all.
-
t at
least such refactoring makes code a little more readable.
--
Best regards,
Aleksander Alekseev
diff --git a/src/backend/parser/parse_type.c b/src/backend/parser/parse_type.c
index a8bb472..a1c5853 100644
--- a/src/backend/parser/parse_type.c
+++ b/src/backend/parser/parse_type.c
@@ -17,6 +
sist on changes like
this since it's a matter of taste.
The bottom line --- this code looks good to me. If there is nothing you
would like to change in the last moment I suggest to change the status
to "Ready for committer".
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
2016 at 03:18:59PM +0300, Aleksander Alekseev wrote:
> > I noticed, that `make installcheck` fails on my laptop with following
> > errors:
> >
> > http://afiskon.ru/s/98/6f94ce2cfa_regression.out.txt
> > http://afiskon.ru/s/b3/d0da05597e_regression.diffs.txt
>
>
Arch Linux and GCC 6.2.1.
Any ideas what can cause this issue?
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
27;s not a case currently.
[1] https://github.com/postgrespro/sr_plan
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
y the only real question here is how it should look like in
> postgresql.conf.
OK, how about introducing a new boolean parameter named log_copy?
Corresponding patch is attached.
--
Best regards,
Aleksander Alekseev
diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
index 8c25b45.
ose. For instance, if DBMS is provided as
a service.
Currently PostgreSQL allows to record all workload _except_ COPY
queries. Considering how easily it could be done I think it's wrong.
Basically the only real question here is how it should look like in
postgresql.conf.
--
Best regards,
Aleksander Alekseev
signature.asc
Description: PGP signature
application
that uses COPY and you don't have access to or something like this.
It could also be useful in some other cases.
This feature is very simple and easy to maintain. I'm sure we could find
a solution that will make happy both developers and users.
--
Best regards,
Aleksan
27;;
LOG: statement: 111aaa
LOG: statement: 222bbb
```
--
Best regards,
Aleksander Alekseev
diff --git a/src/backend/commands/copy.c b/src/backend/commands/copy.c
index 5947e72..82b3a18 100644
--- a/src/backend/commands/copy.c
+++ b/src/backend/commands/copy.c
@@ -331,6 +331
xes Makefile. On some systems (notably FreeBSD) there is
no `gcc` by default. Using `cc` is a more crossplatform way to compile a
project.
--
Best regards,
Aleksander Alekseev
diff --git a/pg_filedump.c b/pg_filedump.c
index 2f2cd53..ba55711 100644
--- a/pg_filedump.c
+++ b/pg_filedump.c
@@ -991,7 +99
what
authorization method was chosen.
Suggested patch (first of many, I hope) renames `md5Salt` to more
general `pwsalt`.
Does it sound reasonable?
--
Best regards,
Aleksander Alekseev
diff --git a/src/backend/libpq/auth.c b/src/backend/libpq/auth.c
index 0ba8530..25bb4c2 100644
--- a/src/backe
f (msg->id == SHAREDINVALRELMAP_ID)
> appendStringInfo(buf, " relmap db %u", msg->rm.dbId);
>
> Looking at inval.c, dbId can be InvalidOid.
Frankly I'm not very familiar with this part of code. InvalidOid is just
zero. Does it create some problem in this case?
--
Best regards,
Alek
Hello.
Very simple small patch - see attachment.
--
Best regards,
Aleksander Alekseev
diff --git a/src/backend/access/rmgrdesc/standbydesc.c b/src/backend/access/rmgrdesc/standbydesc.c
index 13797a3..77983e5 100644
--- a/src/backend/access/rmgrdesc/standbydesc.c
+++ b/src/backend/access
vironments, a not just a hack for single misconfigured
system.
--
Best regards,
Aleksander Alekseev
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
de option
> /home/eax/temp/libpq-failover-8.patch:254: trailing whitespace.
>*/
> /home/eax/temp/libpq-failover-8.patch:306: trailing whitespace.
> appendPQExpBuffer(&conn->errorMessage,
> error: src/interfaces/libpq/t/001-multihost.pl: already exists in
> worki
appendPQExpBuffer(&conn->errorMessage,
error: src/interfaces/libpq/t/001-multihost.pl: already exists in
working directory
$ git diff
```
--
Best regards,
Aleksander Alekseev
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
that
describes how this procedure works, what makes you think that it gives a
good distribution in all possible cases (e.g. if there is more than
0x1 elements in a list - why not), etc. Right? :)
--
Best regards,
Aleksander Alekseev
--
Sent via pgsql-hackers mailing list (pgsql-h
1 - 100 of 229 matches
Mail list logo
