zy-kkk commented on PR #8862:
URL: 
https://github.com/apache/arrow-datafusion/pull/8862#issuecomment-1906262994

   > > Thank you very much @zy-kkk -- I reviewed this PR and I think it looks 
quite good.
   > > It would be nice to add a `position` alias, but also think we can do 
that as a follow on PR as well
   > 
   > Thank you, @alamb, I will continue to modify this PR so that 
`position('world' in 'Helloworld') = 6` can be run to be compatible with 
postgresql
   
   Hi @alamb, I tested changing instr to position. If I want to support the 
syntax `position('world' in 'Helloworld') = 6`, I need to do it at 
https://github.com/sqlparser-rs/sqlparser-rs Add a syntax parsing in , which 
may be like adding the `overlay` function in 
https://github.com/sqlparser-rs/sqlparser-rs/pull/594. What do you suggest?


-- 
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]

Reply via email to