alamb commented on issue #2036: URL: https://github.com/apache/datafusion-sqlparser-rs/issues/2036#issuecomment-3415202345
Thank you @eyalsatori -- this sounds like a great incremental way to move this project along So with this strategy, as I understand it, any existing users of `Token` (aka who need owned tokens) would get `BorrowedToken<'static>` which should be almost as easy to work with. Internally the parser would be able to simply use the Borrowed token that refers to the original input string 👍 If this works out well we can maybe even use the same process in the AST nodes. FYI @iffyio and @yoavcloud I would be happy to help test the approach downstream in DataFusion -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
