virajjasani commented on PR #2199: URL: https://github.com/apache/phoenix/pull/2199#issuecomment-2998866640
> My assumption was that `miniBatchOp.setOperationStatus()` can be called for index `1` to return a new `Result` even if it is for the same row. Are you saying this won't work because they will have the same row key? The index must match the num of operations in the miniBatch, therefore for the single row update, we cannot use index other than 0 IIRC. > Regarding data transfer overhead, I wasn't suggesting that we should return both versions unconditionally. We could take in a param for the caller to indicate what row versions they are interested in. Got it, that's what this PR is doing, but we cannot use any index other than 0 for the single row atomic update. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
