deniskuzZ commented on PR #5513: URL: https://github.com/apache/hive/pull/5513#issuecomment-2442471715
> > before change > > ``` > > "100"|"val_100" > > ``` > > > > > > > > > > > > > > > > > > > > > > > > after change quotes are removed > > ``` > > 100|val_100 > > ``` > > Ah, I see you mean now. It was not my intent to change any original defaults. However, this change was made a long time ago for our private use. It was only ressurrected somewhat recently for our use with Hive/MR3 and I assumed that when working with them we'd kept any old behavior unchanged. I'll fix this as soon as I'm able to test it properly. I have more pressing, immediate needs to attend to first. > > > @gigem, shouldn't we apply the quotes by default if `quoteChar` is provided?? > > I don't think so. The desire to quote or not quote all columns can be independtent of the actual, quote character. in that case, you could simply add `applyQuotesToAll`='true' in `separator_test` table serde properties -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
