uninvert test for coordinator node / primary field translation node

If we're the primary field translation node, we don't need to set up
translation replication; we only need that if we're *not*. So it
makes sense to test if !IsPrimaryFieldTranslationNode... except that
the test is to determine whether to return early. So it should not
be inverted.
This commit is contained in:
Seebs 2021-02-12 15:30:54 -06:00
parent 0790fbe866
commit e2b6912d1c

View file

@ -1947,7 +1947,7 @@ func (s *holderSyncer) initializeIndexTranslateReplication(snap *topology.Cluste
// initializeFieldTranslateReplication connects the coordinator to stream field data.
func (s *holderSyncer) initializeFieldTranslateReplication(snap *topology.ClusterSnapshot) error {
// Skip if coordinator.
if !snap.IsPrimaryFieldTranslationNode(s.Cluster.Node.ID) {
if snap.IsPrimaryFieldTranslationNode(s.Cluster.Node.ID) {
return nil
}