Clarification: UpdateResult in SetResponse
gsindigi opened this issue · 0 comments
gsindigi commented
SetResponse contains repeated UpdateResult response = 2;
; however, UpdateResult
's fields timestamp
and message (Error)
fields are deprecated. Remaining two fields are to carry the path and Operation
details, which are pretty much dependent on incoming SetRequest
. These were making sense when each element was carrying the error information if there's any. Given this, should UpdateResult
continue to be part of SetResponse
? Or, should it also be considered to deprecate this as well? As the error reporting is part of the RPC return, this UpdateResult
in response
may not carry any useful information.