Align old and new client resp handling

Signed-off-by: Matt Lord <[email protected]>
vitessio · Jan 1, 2025 · 3018fa3 · 3018fa3
1 parent 82c335a
commit 3018fa3
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/go/cmd/vtctldclient/command/schema.go b/go/cmd/vtctldclient/command/schema.go
@@ -388,6 +388,10 @@ func commandValidateSchemaKeyspace(cmd *cobra.Command, args []string) error {
 		return err
 	}
 
+	if len(resp.Results) > 0 {
+		return fmt.Errorf("%s", resp.Results[0])
+	}
+
 	data, err := cli.MarshalJSON(resp)
 	if err != nil {
 		return err

diff --git a/go/vt/vtctl/grpcvtctldserver/server.go b/go/vt/vtctl/grpcvtctldserver/server.go
@@ -4801,6 +4801,7 @@ func (s *VtctldServer) ValidateSchemaKeyspace(ctx context.Context, req *vtctldat
 	defer panicHandler(&err)
 
 	span.Annotate("keyspace", req.Keyspace)
+	span.Annotate("shards", req.Shards)
 	keyspace := req.Keyspace
 
 	resp = &vtctldatapb.ValidateSchemaKeyspaceResponse{