topo: Clean up unused code (#15515)

Signed-off-by: Dirkjan Bussink <d.bussink@gmail.com>

go/vt/topo/consultopo/version.go

@@ -18,8 +18,6 @@ package consultopo
 
 import (
 	"fmt"
-
-	"vitess.io/vitess/go/vt/topo"
 )
 
 // ConsulVersion is consul's idea of a version.
@@ -31,13 +29,3 @@ type ConsulVersion uint64
 func (v ConsulVersion) String() string {
 	return fmt.Sprintf("%v", uint64(v))
 }
-
-// VersionFromInt is used by old-style functions to create a proper
-// Version: if version is -1, returns nil. Otherwise returns the
-// ConsulVersion object.
-func VersionFromInt(version int64) topo.Version {
-	if version == -1 {
-		return nil
-	}
-	return ConsulVersion(version)
-}

go/vt/topo/etcd2topo/version.go

@@ -18,8 +18,6 @@ package etcd2topo
 
 import (
 	"fmt"
-
-	"vitess.io/vitess/go/vt/topo"
 )
 
 // EtcdVersion is etcd's idea of a version.
@@ -31,13 +29,3 @@ type EtcdVersion int64
 func (v EtcdVersion) String() string {
 	return fmt.Sprintf("%v", int64(v))
 }
-
-// VersionFromInt is used by old-style functions to create a proper
-// Version: if version is -1, returns nil. Otherwise returns the
-// EtcdVersion object.
-func VersionFromInt(version int64) topo.Version {
-	if version == -1 {
-		return nil
-	}
-	return EtcdVersion(version)
-}