itschrispeck commented on issue #12433: URL: https://github.com/apache/pinot/issues/12433#issuecomment-1953523983
Hi @abhioncbr, this is an interesting bottleneck. So I understand - by skipping the unescaping step this will prevent column names from containing either `:` or `=`, right? I was hoping Pinot would eventually support these characters in column names officially as most other DBs do (I had started some [PR](https://github.com/apache/pinot/pull/12018), but didn't get around to refactoring based on feedback). Unfortunately, we're running an internal patch and supporting hundreds of tables with these special columns already. It sounds like to support these names in the future/to avoid breaking our internal branch we'd have to make this reader pluggable and support a version that does allow for escaping keys? -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
