eric-maynard commented on PR #1990: URL: https://github.com/apache/polaris/pull/1990#issuecomment-3050918650
+1 @flyrain, your comment reminds me of the discussion on [this other PR](https://github.com/apache/polaris/pull/1899#issuecomment-2977562907). Unfortunately right now the `...Entity...` types are _not_ purely persistence concerns. It might be nice if that were the case, but that would require introduction of some new business types (or DAOs) first. However, I don't think having converters is the worst idea, at least it will centralize some of the management of these objects and allow for future improvements like the introduction of new intermediary types. -- 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: issues-unsubscr...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
