eric-maynard commented on PR #1899: URL: https://github.com/apache/polaris/pull/1899#issuecomment-2977562907
For better or worse, we currently use entities both for the persisted state of an object and for an in-memory representation of the same. @snazy perhaps we can view this as a step in the right direction as it encourages us to treat entities as immutable and to use discrete "transformations" to generate new entities instead of trying to modify them in place or to construct lots of new entities in an adhoc way throughout the code base. Eventually, entities should be truly immutable and this may become the canonical way to transform an entity into a new one. I am, however, a little concerned about how heavyweight this process could become... if I just want to update an entity's version today, I can call a method like `setGrantRecordsVersion`. After this change, it looks like the expectation is that I have a transformation engine, and then I construct and execute a transformation to accomplish this change? -- 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]
