so, I thinks that hash table is not very good option if we have all pages
there: we should either use array or do not fill table for every page.
If author and community do not object, I want to continue work on Konstantin's
patch.
Best regards, Andrey Borodin.
0001-GiST-VACUUM-rebase.patch
De
gnoring this for current version.
2. I've done grammarly checking :) This comma seems redundant [2]
I don't think something of these items require fixing.
Thanks for working on this, I believe it is important.
Best regards, Andrey Borodin.
[0] https://github.com/x4m/pgscripts/blob/master/install
gt;
> Thus, it should be safe to just remove both compress/decompress methods from
> existing opclass.
Alexander, Tom, you are absolutely right. I was sure there is toasting code in
cube's compress, but it was not ever there.
Here is patch for cube that drops functions.
Best regards, Andrey Bo
nalization.
But from your words I see that the safest option is to check timeline from
pg_control after start and after stop. If this timelines differ - invalidate
backup entirely. This does not seem too hard condition for invalidation, does
it?
Best regards, Andrey Borodin.
--
Sent via pgsql-hackers
Hi, hackers!
I'm working on backups from replication salve in WAL-G [0]
Backups used to use result of pg_walfile_name(pg_start_backup(...)). Call to
pg_start_backup() works nice, but "pg_walfile_name() cannot be executed during
recovery."
This function has LSN as argument and reads TimeLineId
Hi hackers!
Here are patches enabling Index Only Scan for cube and seg extensions.
These patches follow this discussion [0].
For cube there is new default opclass. We cannot drop old opclass, because it
could TOAST come cube values in rare occasions. Index Only Scan is enabled only
for newly
> 22 окт. 2017 г., в 21:21, Tom Lane <t...@sss.pgh.pa.us> написал(а):
>
> Andrey Borodin <x4...@yandex-team.ru> writes:
>> I was looking for a way to correctly drop compress\decompress functions from
>> opclasses.
>
> Making a new opclass seems like
Hi, Tom!
> 20 сент. 2017 г., в 8:38, Tom Lane <t...@sss.pgh.pa.us> написал(а):
>
> Andrey Borodin <x4...@yandex-team.ru> writes:
>> [ 0001-Allow-uncompressed-GiST-4.patch ]
>
> Pushed, with a bit more work on the documentation and some minor
> cosmetic c
Hello!
> 9 окт. 2017 г., в 10:23, Andrey Borodin <x4...@yandex-team.ru> написал(а):
>
> Thanks, Stephen, this actually pointed what to look for
> VM is WAL-logged [0]
> FSM is not [1]
>
> [0]
> https://github.com/postgres/postgres/blob/113b0045e20d40f726a0a3
copied page-incrementally, FSM cannot.
But the design you are describing resembles PTRACK[0]: fork for page changes
tracking, not WAL-logged, but crash safe due to recovery from others WALs.
Best regards, Andrey Borodin.
[0] https://gist.github.com/stalkerg/ab833d94e2f64
> 8 окт. 2017 г., в 20:11, Stephen Frost <sfr...@snowman.net> написал(а):
> * Andrey Borodin (x4...@yandex-team.ru) wrote:
>> But my other question still seems unanswered: can I use LSN logic for
>> incrementing FSM and VM? Seems like most of the time there is valid
by Stephen are taken.
But my other question still seems unanswered: can I use LSN logic for
incrementing FSM and VM? Seems like most of the time there is valid LSN
* I like the idea of using something for both incr(diff) backups and VACUUM, it
worth thinking about.
Best regards, Andrey Borodin
pacity" of FS?
Thank you for your attention. I'll be glad to receive any information\pointers
on this matter.
Best regards, Andrey Borodin, Yandex.
[0] https://github.com/wal-g/wal-g/pull/29
[1]
https://github.com/wal-g/wal-g/pull/29/files#diff-d77406e827f5f947d4d4a1e6d76c1f4eR114
[2]
https
Hello Tom! Thanks for committing the patch!
> 20 сент. 2017 г., в 8:38, Tom Lane <t...@sss.pgh.pa.us> написал(а):
>
> Andrey Borodin <x4...@yandex-team.ru> writes:
>> [ 0001-Allow-uncompressed-GiST-4.patch ]
>
> ... There's still room
> to improve the con
ons to be
omitted) Thanks, Tom! : )
"Index Only Scan support for cube" patch now is obsolete. I'm working on
another similar patch for contribs to support GiST IOS and remove no-op support
functions.
Best regards, Andrey Borodin.
--
Sent via pgsql-hackers mailing list (pgsql-hackers@pos
.
> BTW, this change in the patch look suspicious for me
>
> We are typically evade changing formatting in fragments of codes not directly
> touched by the patch.
Thanks for spotting this out. Here's fixed version.
Best regards, Andrey Borodin.
0001-Allow-uncompressed-GiST
Hi!
Thank you for your interest and experiment results.
> 13 сент. 2017 г., в 15:43, Ants Aasma <ants.aa...@eesti.ee> написал(а):
>
> On Thu, Aug 31, 2017 at 9:02 AM, Andrey Borodin <x4...@yandex-team.ru> wrote:
>> When we have accumulated diff blocknumbers
end to S3. And there
is no system with CPUs that can handle that WAL parsing. Currently, I'm
considering mocking prototype for wal-g, which works exactly this.
Your comments were very valuable, thank you for looking into the patch and
joining the discussion.
Best regards, Andrey Borodin.
ment. Thanks for looking into the patch!
Best regards, Andrey Borodin.
0001-Allow-uncompressed-GiST-3.patch
Description: Binary data
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
ut it more. This proposal takes vanishingly small part of
backend performance, but, indeed, nonzero part.
Again, thank you for your time and comments.
Best regards, Andrey Borodin.
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
nable more efficient backups. How do you
think?
Any ideas will be appreciated. This patch is influenced by the code of PTRACK
(Yury Zhuravlev and Postgres Professional).
Best regards, Andrey Borodin.
[0]
https://www.postgresql.org/message-id/flat/20051502087457%40webcorp01e.yandex-team.ru#2
h critical section. But I want to avoid intervention into
critical sections.
Thank you for your attention, any thoughts will be appreciated.
Best regards, Andrey Borodin.
[0] https://gist.github.com/stalkerg/ab833d94e2f64df241f1835651e06e4b
0001-hooks-to-watch-for-changed-pages.patch
Descript
correct checklist:
installcheck-world passes, documented as expected. Feature is there.
Best regards, Andrey Borodin.
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
actual
values, but only if it is on first index page", like conditional locks. But I
think this patch is a step to right direction.
Best regards, Andrey Borodin.
--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
int deserves special attention)
1. Patch applies and installcheck-world passes
2. Patch contains new isolation tests
3. These tests fail if indexes do not implement predicate locking
4. Patch updates documentation
Shubham also had checked that patches work (install check-world) on 1Kb, 8Kb
and 32Kb pag
The algorithm used by
> BufferSync to schedule buffer writes has changed multiple times
> before and doubtless will again; if we're going to have a hook
> here it should depend as little as possible on those details.
OK, now I see that «buf_internals.h» had word internals for a reason. Than
.
Summarizing these points, appropriate statuses of the patch are ‘Ready for
committer’ or ‘Rejected’. Between these two I choose ‘Ready for committer’, I
think patch is committable (after bg sessions).
Best regards, Andrey Borodin.
The new status of this patch is: Ready for Committer
--
Sent via pgsql
patches, but this review does not apply to them. I
suppose db_link refactoring would be useful and functionality is added, so I
think these patches deserve separate commitfest entry.
Best regards, Andrey Borodin.
The new status of this patch is: Ready for Committer
--
Sent via pgsql-hackers
Thank you for your attention to details, Mikhail.
pack_float_good() looks good. But I'm not sure inline strict init is allowed
under ansi C. Converting to regular ancient form b.fp = v; won't change compile
result, would it?
Regards, Andrey Borodin.
--
Sent via pgsql-hackers mailing list
That storage assertion fired during usual update table set x=random() without
conditions. Also Make check fails without it (for brin usage, gist is ok with
it).
Cannot type quotation properly, I'm on a phone for a long time.
Regards, Andrey Borodin.
--
Sent via pgsql-hackers mailing list
Index size is in the middle between full index and minimal index.
I think this numbers agree with expectation from the feature.
CONCLUSION==
This patch brings useful and important feature. Build shall be repaired; other
my suggestions are only suggestions.
Best regards, Andrey
> 3 авг. 2016 г., в 22:47, Andrey Borodin <amboro...@acm.org> написал(а):
>
> make installcheck-world: tested, failed
> Implements feature: tested, failed
> Spec compliant: tested, failed
> Documentation:tested, failed
>
> This is a
The following review has been posted through the commitfest application:
make installcheck-world: tested, failed
Implements feature: tested, failed
Spec compliant: tested, failed
Documentation:tested, failed
This is a review of a patch "Optimizing numeric SUM()
33 matches
Mail list logo