[GitHub] [calcite] amaliujia commented on pull request #2113: [CALCITE-4176] Key descriptor can be optional in SESSION table function (Rui Wang)

2020-09-25 Thread GitBox
amaliujia commented on pull request #2113: URL: https://github.com/apache/calcite/pull/2113#issuecomment-699007258 tracking in https://github.com/apache/calcite/pull/2164 This is an automated message from the Apache Git

[GitHub] [calcite] amaliujia commented on pull request #2113: [CALCITE-4176] Key descriptor can be optional in SESSION table function (Rui Wang)

2020-09-09 Thread GitBox
amaliujia commented on pull request #2113: URL: https://github.com/apache/calcite/pull/2113#issuecomment-689993485 @danny0405 Got really busy recently and I can try to finish this one before next release. If there is an urgency that requires this change, I might be able to pick

[GitHub] [calcite] amaliujia commented on pull request #2113: [CALCITE-4176] Key descriptor can be optional in SESSION table function (Rui Wang)

2020-08-18 Thread GitBox
amaliujia commented on pull request #2113: URL: https://github.com/apache/calcite/pull/2113#issuecomment-675836435 R: @danny0405 Do you have any suggestions on CALCITE-4176? Espeically the idea to make the argument list of SESSION table function as: `data, timecol, key optional,