Return mutated uids as part of the api.Response

[pawanrawal]

This would allow Alpha nodes to return a map of variable name => the uids belonging to the variable.

---

This change is 

[pullrequest]

✅ A review job has been created and sent to the PullRequest network.

---

@pawanrawal you can click here to see the review status or cancel the code review job.

[MichaelJCompton]

Looks fine to me. I'll leave other comments to @mangalaman93 as he knows the upsert code.

Reviewable status: 0 of 2 files reviewed, all discussions resolved (waiting on @mangalaman93)

[manishrjain]

Let’s avoid duplicated, similar looking fields.

Reviewable status: 0 of 2 files reviewed, 1 unresolved discussion (waiting on @mangalaman93 and @pawanrawal)

---

protos/api.proto, line 66 at r1 (raw file):

	Latency latency = 3;
	map<string, Uids> mutated = 4;
	map<string, string> uids = 12;

Why not make uids field return <string, Uids> instead of having two which look similar.

[pullrequest]

The change looks good to me and adds the UID map

---

Reviewed with ❤️ by PullRequest

[mangalaman93]

Reviewed 2 of 2 files at r2.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @pawanrawal)