2
0

Merge pull request #389 from steampunkcoder/FixStartReplicationSyntaxError

Fix syntax error in replication protocol msg issued by StartReplication()
This commit is contained in:
Jack Christensen
2018-02-05 16:20:27 -06:00
committed by GitHub
+6 -6
View File
@@ -4,6 +4,7 @@ import (
"context" "context"
"encoding/binary" "encoding/binary"
"fmt" "fmt"
"strings"
"time" "time"
"github.com/pkg/errors" "github.com/pkg/errors"
@@ -401,15 +402,14 @@ func (rc *ReplicationConn) TimelineHistory(timeline int) (r *Rows, err error) {
// This function assumes that slotName has already been created. In order to omit the timeline argument // This function assumes that slotName has already been created. In order to omit the timeline argument
// pass a -1 for the timeline to get the server default behavior. // pass a -1 for the timeline to get the server default behavior.
func (rc *ReplicationConn) StartReplication(slotName string, startLsn uint64, timeline int64, pluginArguments ...string) (err error) { func (rc *ReplicationConn) StartReplication(slotName string, startLsn uint64, timeline int64, pluginArguments ...string) (err error) {
var queryString string queryString := fmt.Sprintf("START_REPLICATION SLOT %s LOGICAL %s", slotName, FormatLSN(startLsn))
if timeline >= 0 { if timeline >= 0 {
queryString = fmt.Sprintf("START_REPLICATION SLOT %s LOGICAL %s TIMELINE %d", slotName, FormatLSN(startLsn), timeline) timelineOption := fmt.Sprintf("TIMELINE %d", timeline)
} else { pluginArguments = append(pluginArguments, timelineOption)
queryString = fmt.Sprintf("START_REPLICATION SLOT %s LOGICAL %s", slotName, FormatLSN(startLsn))
} }
for _, arg := range pluginArguments { if len(pluginArguments) > 0 {
queryString += fmt.Sprintf(" %s", arg) queryString += fmt.Sprintf(" ( %s )", strings.Join(pluginArguments, ", "))
} }
if err = rc.c.sendQuery(queryString); err != nil { if err = rc.c.sendQuery(queryString); err != nil {