[GitHub] [arrow] alamb commented on pull request #9108: ARROW-11138: [Rust] Add ltrim, rtrim to built-in functions

2021-01-07 Thread GitBox
alamb commented on pull request #9108: URL: https://github.com/apache/arrow/pull/9108#issuecomment-756195398 Filed https://issues.apache.org/jira/browse/ARROW-11164 for the optional second argument Filed https://issues.apache.org/jira/browse/ARROW-11165 to document the desired diale

[GitHub] [arrow] alamb commented on pull request #9108: ARROW-11138: [Rust] Add ltrim, rtrim to built-in functions

2021-01-06 Thread GitBox
alamb commented on pull request #9108: URL: https://github.com/apache/arrow/pull/9108#issuecomment-755777874 I agree with @seddonm1 that ideally we would follow Postgres when possible so we can just re-use all their documentation and semantics and so we don't end up inadvertently creating