eric-maynard commented on PR #247: URL: https://github.com/apache/polaris/pull/247#issuecomment-2327609623
While this works, I don't see an easy way to ensure compatibility with metastores already bootstrapped using the old code. We need to respect the `last_value` of existing sequences, but make the code robust to situations where the sequence doesn't (or can't) exist. Adding a try/catch to this hot code path seems unwise, so I'll spend more time workshopping a fix. -- 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]
