dimas-b commented on code in PR #4269:
URL: https://github.com/apache/polaris/pull/4269#discussion_r3235557066
##########
polaris-core/src/main/java/org/apache/polaris/core/entity/IdempotencyRecord.java:
##########
@@ -50,115 +52,17 @@ public record IdempotencyRecord(
String idempotencyKey,
String operationType,
String normalizedResourceId,
+ String principalHash,
Review Comment:
Since this class existed before, and current PR concentrates on integrating
idempotency support into Polaris services, would you mind extracting functional
changes into a new PR to be merged before this PR?
I believe this will make reviews easier by isolating functional changes from
infrastructural changes.
--
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]
