eric-maynard commented on issue #621: URL: https://github.com/apache/polaris/issues/621#issuecomment-2597649376
Yes -- if POLARIS_SEQ (removed [here](https://github.com/apache/polaris/commit/537eee6578afbea8dede47958edf0b6168326252#diff-86c8895dd16cf603f1de2aa49931368a075f7aafbe2bc4fca0bc7b8c2aa9be76L68)) was used in a release then I would prefer this comment to say something like: ``` If the generator `POLARIS_SEQ` used prior to Polaris X.Y is... ``` But since there isn't a version, "legacy" was the most descriptive term I could think of. The generator is legacy in that it may still be around in the metastore but should only be used in a one-off operation and otherwise is disused now. Eventually I think we can even clean a lot of this up. We just needed this to make sure we didn't break any existing deployments using the old (legacy) metastore implementation. -- 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]
