add bidirectional streaming support for external snapshot streaming

[shivaji-kharse]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces bidirectional streaming for external snapshot functionality. Key changes include replacing unary response methods with stream-based ones in generated gRPC code, adding a new "finish" field to the snapshot response message, and updating the proto service definition to use bidirectional streaming.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
protos/api.v2/api.v2_grpc.pb.go	Replaces CloseAndRecv/SendAndClose with Recv/Send for bidirectional streaming and updates ServiceDesc accordingly.
protos/api.v2/api.v2.pb.go	Adds a new "Finish" boolean field and its getter to the snapshot response message.
protos/api.v2.proto	Updates the RPC definition to switch from a unary to bidirectional streaming response.

Comments suppressed due to low confidence (3)

protos/api.v2/api.v2_grpc.pb.go:181

• Replacing CloseAndRecv with Recv may require clients to explicitly call CloseSend() to terminate the send stream. Ensure that this change is clearly communicated in the documentation to prevent resource leaks.

	Recv() (*StreamExtSnapshotResponse, error)

protos/api.v2/api.v2.pb.go:1515

• [nitpick] Consider adding documentation or code comments to clarify the intended semantics of the new 'Finish' field to facilitate its usage on the client side.

	Finish        bool                   `protobuf:"varint,1,opt,name=finish,proto3" json:"finish,omitempty"`

protos/api.v2.proto:34

• Changing the RPC signature to bidirectional streaming is a breaking change; ensure that clients are updated accordingly and that compatibility guidelines are documented.

  rpc StreamExtSnapshot(stream StreamExtSnapshotRequest) returns (stream StreamExtSnapshotResponse) {}