On 8 February 2014 19:35, Thom Brown <thom@linux.com> wrote:
> On 8 February 2014 17:52, Andres Freund <andres@2ndquadrant.com> wrote:
>> Hi,
>>
>> Only got to this now, was a bit too tired and needed to catch up on some
>> real-world stuff...
>>
>> On 2014-02-08 00:16:07 +0000, Thom Brown wrote:
>>> On 7 February 2014 23:43, Andres Freund <andres@2ndquadrant.com> wrote:
>>> > Thanks, that's a bug indeed. I have experimentally fixed the bug, not
>>> > sure whether I like the fix yet, or not.
>>> >
>>> > I've already fixed two issues caused by the rebase onto
>>> > 858ec11858a914d4c380971985709b6d6b7dd6fc.
>>> >
>>> > Is pushing to git sufficient for you, or shall I rebase and resend the
>>> > series?
>>>
>>> Sure, push it to git, I'll add your remote repo and checkout that branch.
>>
>> Ok, I roughly went with my initial plan to fix this and I've added (and
>> fixed) a regression for this.
>>
>> Pushed this and some other improvements to
>> http://git.postgresql.org/gitweb/?p=users/andresfreund/postgres.git;a=summary
>> branch xlog-decoding-rebasing-remapping
>
> This appears to be working now. Thanks.
>
> I'll continue to play around with the feature.
Next issue. Firstly, an out-of-date example:
doc/src/sgml/changesetextraction.sgml
pg_recvlogical --slot test --init -d testdb
There's no option --init. I think this is supposed to be --create.
But also:
$ pg_recvlogical --slot test --create -d testdb
pg_recvlogical: could not send replication command
"CREATE_REPLICATION_SLOT "test" LOGICAL "test_decoding"": extraneous
data in "T" message
But this seems to have created it anyway:
# SELECT * FROM pg_replication_slots;slot_name | plugin | slot_type | datoid | database | active |
xmin | catalog_xmin | restart_lsn
-----------+---------------+-----------+--------+----------+--------+------+--------------+-------------test |
test_decoding| logical | 16396 | testdb | t | | | 0/16E2030
(1 row)
If I drop it and run the same command, the same message is emitted.
--
Thom