dimas-b commented on code in PR #3200:
URL: https://github.com/apache/polaris/pull/3200#discussion_r2590308772
##########
polaris-core/src/main/java/org/apache/polaris/core/persistence/dao/entity/EntityResult.java:
##########
@@ -87,15 +85,6 @@ public PolarisEntitySubType getAlreadyExistsEntitySubType() {
}
}
- @JsonCreator
- private EntityResult(
Review Comment:
Thanks for the info, @HonahX !
However, from the OSS perspective this use case is absolutely not apparent
:shrug: Adding tests can be a means for avoiding regressions in downstream, but
the situation is still that a downstream project imposes a restriction on
Polaris code in this case without any OSS use cases.
@flyrain : What is your take on this in light of the recent [dev ML
conversation](https://lists.apache.org/thread/ybq0wjyf741txbpw56gzbzx0hpf9b4w3)?
I propose to use this PR as an illustration and continue the discussion of
`dev`.
--
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]
