Re: [PR] [chore][ci] add README.md for regression-test [doris]

2023-11-23 Thread via GitHub
hello-stephen commented on PR #26876: URL: https://github.com/apache/doris/pull/26876#issuecomment-1825067264 3. 要考虑case多次运行的情况,case第一次运行成功了,但是残留了一些东西导致再次运行时失败。比如,残留了table,resource,policy,导致下一次create失败 -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] [chore][ci] add README.md for regression-test [doris]

2023-11-19 Thread via GitHub
hello-stephen commented on PR #26876: URL: https://github.com/apache/doris/pull/26876#issuecomment-1818190122 2. sql 结果给每列都要添加order by, 以免结果未排序导致的对比失败 -- 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

Re: [PR] [chore][ci] add README.md for regression-test [doris]

2023-11-14 Thread via GitHub
zclllyybb commented on PR #26876: URL: https://github.com/apache/doris/pull/26876#issuecomment-1809706206 涉及到函数的测试,应该覆盖四种case:NULL列,NOT NULL列,字面量,常量折叠(如有) 对于涉及到datev2与datetimev2相关的,为了确定函数参数推导正确,应该覆盖字符串只有date格式、有datetimev2格式(小数0位,小数超过6位)的情况。如果老优化器实在有问题难以修复的可以忽略,保证新优化器行为正确。 -- This is

Re: [PR] [chore][ci] add README.md for regression-test [doris]

2023-11-13 Thread via GitHub
hello-stephen commented on PR #26876: URL: https://github.com/apache/doris/pull/26876#issuecomment-1809672554 补充: 1. 写case时候 要考虑3fe 4be 和 1fe 1be情况 -- 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

Re: [PR] [chore][ci] add README.md for regression-test [doris]

2023-11-13 Thread via GitHub
zhangstar333 merged PR #26876: URL: https://github.com/apache/doris/pull/26876 -- 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:

Re: [PR] [chore][ci] add README.md for regression-test [doris]

2023-11-13 Thread via GitHub
github-actions[bot] commented on PR #26876: URL: https://github.com/apache/doris/pull/26876#issuecomment-1808329718 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] [chore][ci] add README.md for regression-test [doris]

2023-11-13 Thread via GitHub
github-actions[bot] commented on PR #26876: URL: https://github.com/apache/doris/pull/26876#issuecomment-1808329624 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] [chore][ci] add README.md for regression-test [doris]

2023-11-13 Thread via GitHub
hello-stephen commented on PR #26876: URL: https://github.com/apache/doris/pull/26876#issuecomment-1808071244 run buildall -- 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.

Re: [PR] [chore][ci] add README.md for regression-test [doris]

2023-11-12 Thread via GitHub
hello-stephen commented on PR #26876: URL: https://github.com/apache/doris/pull/26876#issuecomment-1807592091 run buildall -- 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.

[PR] [chore][ci] add README.md for regression-test [doris]

2023-11-12 Thread via GitHub
hello-stephen opened a new pull request, #26876: URL: https://github.com/apache/doris/pull/26876 ## Proposed changes Issue Number: close #xxx ## Further comments If this is a relatively large or complex change, kick off the discussion at