Re: [I] Question on: `visit_expressions_mut` for alias expr [datafusion-sqlparser-rs]

2025-06-05 Thread via GitHub
HuyNguyen7994 commented on issue #1475: URL: https://github.com/apache/datafusion-sqlparser-rs/issues/1475#issuecomment-2944295472 @cisaacson I'm having the same problem. Turn out you want to visit `SelectItem`: https://github.com/HuyNguyen7994/datafusion-sqlparser-rs/tree/add-select-item-

Re: [I] Question on: `visit_expressions_mut` for alias expr [datafusion-sqlparser-rs]

2025-01-25 Thread via GitHub
docteurklein commented on issue #1475: URL: https://github.com/apache/datafusion-sqlparser-rs/issues/1475#issuecomment-2613924635 here is an example to change ASC to DESC in some order by: ```rust struct VisitOrderBy { reorder: Vec, } impl VisitorMut for VisitOrde

Re: [I] Question on: `visit_expressions_mut` for alias expr [datafusion-sqlparser-rs]

2025-01-23 Thread via GitHub
docteurklein commented on issue #1475: URL: https://github.com/apache/datafusion-sqlparser-rs/issues/1475#issuecomment-2610324878 Given what I see here https://github.com/openobserve/openobserve/blob/ebcb12c4ead3c31d16f2ef5f515707c670dd2611/src/service/search/sql.rs#L1210, maybe the better

Re: [I] Question on: `visit_expressions_mut` for alias expr [datafusion-sqlparser-rs]

2025-01-23 Thread via GitHub
cisaacson commented on issue #1475: URL: https://github.com/apache/datafusion-sqlparser-rs/issues/1475#issuecomment-2609988461 If someone can provide some guidance on this I would be happy to work on a PR to fix this. -- This is an automated message from the Apache Git Service. To respon

Re: [I] Question on: `visit_expressions_mut` for alias expr [datafusion-sqlparser-rs]

2025-01-23 Thread via GitHub
docteurklein commented on issue #1475: URL: https://github.com/apache/datafusion-sqlparser-rs/issues/1475#issuecomment-2609963101 I think I have the same problem with: ```sql select * from (select 'test' order by 1 asc) ``` It seems impossible to write a `visit_statements