Hi, While I was reviewing one of the logical decoding features, I found Streaming and binary options were missing in tab completion for the alter subscription set option, the attached patch has the changes for the same. Thoughts?
Regards, Vignesh
From 84a0c07760c913576ef38d74ab37ffd9ee3ad686 Mon Sep 17 00:00:00 2001 From: vignesh <vignesh21@gmail.com> Date: Fri, 14 May 2021 11:37:05 +0530 Subject: [PATCH v1] Added missing tab completion for alter subscription set option. Streaming and binary options were missing in tab completion for alter subscription set option, included it. --- src/bin/psql/tab-complete.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/bin/psql/tab-complete.c b/src/bin/psql/tab-complete.c index 6598c5369a..2c15d8064b 100644 --- a/src/bin/psql/tab-complete.c +++ b/src/bin/psql/tab-complete.c @@ -1665,7 +1665,7 @@ psql_completion(const char *text, int start, int end) COMPLETE_WITH("(", "PUBLICATION"); /* ALTER SUBSCRIPTION <name> SET ( */ else if (HeadMatches("ALTER", "SUBSCRIPTION", MatchAny) && TailMatches("SET", "(")) - COMPLETE_WITH("slot_name", "synchronous_commit"); + COMPLETE_WITH("binary", "slot_name", "synchronous_commit", "streaming"); /* ALTER SUBSCRIPTION <name> SET PUBLICATION */ else if (HeadMatches("ALTER", "SUBSCRIPTION", MatchAny) && TailMatches("SET", "PUBLICATION")) { -- 2.25.1