[GitHub] [calcite] wenhuitang commented on issue #1862: [CALCITE-3864] Implement Concat function

2020-03-27 Thread GitBox
wenhuitang commented on issue #1862: [CALCITE-3864] Implement Concat function URL: https://github.com/apache/calcite/pull/1862#issuecomment-604938227 @DonnyZone @danny0405 @chunweilei Thanks a lot. I have addressed all the comments. And I'm ok with "Supports CONCAT for variable arguments",

[GitHub] [calcite] wenhuitang commented on issue #1862: [CALCITE-3864] Implement Concat function

2020-03-25 Thread GitBox
wenhuitang commented on issue #1862: [CALCITE-3864] Implement Concat function URL: https://github.com/apache/calcite/pull/1862#issuecomment-603802313 Comments has been addressed, and reference.md has been updated too. This is