-
Notifications
You must be signed in to change notification settings - Fork 3.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
sql/logictest: TestLogic failed #60824
Comments
(sql/logictest).TestLogic failed on master@ab07ae1da6fe154eebc6c6847cca58b30184e14f:
MoreParameters:
Related:
See this test on roachdash |
The first report seems possibly related to #59340. |
(sql/logictest).TestLogic failed on master@bc98209ed36ec47675fe3d7a7d8201b4b77f89ba:
MoreParameters:
Related:
See this test on roachdash |
(sql/logictest).TestLogic failed on master@5cd07e02c291d017bedd2a237956a62eb63dbcc9:
MoreParameters:
Related:
See this test on roachdash |
Same failure like the one just above: https://teamcity.cockroachdb.com//viewLog.html?buildId=2716122&buildTypeId=Cockroach_MergeToMaster |
(sql/logictest).TestLogic failed on master@ac25d9f3511435c4e6962f71eb169fd6d7d0ef28:
MoreParameters:
Related:
See this test on roachdash |
(sql/logictest).TestLogic failed on master@dcc8ab04251aa954f086cd2b347cd1d1b976a312:
MoreParameters:
Related:
See this test on roachdash |
(sql/logictest).TestLogic failed on master@6521a8e138c9838853f9502303faf99da69aad2d:
MoreParameters:
Related:
See this test on roachdash |
(sql/logictest).TestLogic failed on master@ae1e417e061cc1b869927959e8830e0e5cde2622:
MoreParameters:
Related:
See this test on roachdash |
(sql/logictest).TestLogic failed on master@f0c2d8a2f762ee33ca044ae86f46d06fc35682db:
MoreParameters:
Related:
See this test on roachdash |
(sql/logictest).TestLogic failed on master@a69e6549a71f5a0e83eb13509001f4d7351050fb:
MoreParameters:
Related:
See this test on roachdash |
Hmmm... @ajwerner |
Hmm indeed |
(sql/logictest).TestLogic failed on master@89c379950cb4ca77c5d25e45aac4008b1f1758d9:
MoreParameters:
Related:
See this test on roachdash |
seems to keep happening, and also on release-21.1 |
(sql/logictest).TestLogic failed on master@dd87d114508b8b34230ae7da560ab053dd4f58ce:
MoreParameters:
Related:
See this test on roachdash |
(sql/logictest).TestLogic failed on master@3bae381e45397225ff926f3f8f60d52b70fdc661:
MoreParameters:
Related:
See this test on roachdash |
@jordanlewis test-infra is seeing this and #61657 flake often on master and 21.1. Just wondering if someone is actively fixing this, or you recommend skipping some of these tests? |
@ajwerner please advise. I think we could skip the |
Ack, turning my attention to it now. My best guess is it is #61345 as that's the only real thing that's touched this code that I can think of. |
The code in cockroachdb#61345 added a test that, under stress, nicely revealed a bug. The bug is that we're mutating the AST in place and then, on retries, we hit an error. The fix is to not mutate the AST. I wish I had a more comprehensive testing strategy to ensure this didn't happen. On some level, we could clone the AST when passing it to various DDL plan node constructors. That's very defensive but also probably fine. Another thing that would be cool would be to just assert that after planning the AST did not change. Touches cockroachdb#60824. Release note: None
sql/logictest.TestLogic failed with artifacts on master @ 4d987b3f0788c98fa358715b336a1c55e254ee50:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ 660df848fcd8569392791f35b27ef5e849b35e2c:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ bf6da8a40f6956bc88ee4a930ddc931f73dd5ce4:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ 1c103726de6994b26706244144d3d9f7dd6b1f58:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ bea4754bdf607ab050a72b7e2a3fe9dca0289d34:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ b4766e0bdb491de6020d5d2acaa3d20f9c339176:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ 924935cc8ee14ff0c6d23aedd2a12673bf864808:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
This comment has been minimized.
This comment has been minimized.
sql/logictest.TestLogic failed with artifacts on master @ 588d9d7ad7618784ee23a5e5f440a9d9d36bb0d5:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ 29305bfb08116f38e10ca090347eea170899b919:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ f3286c1e1b8d2d44f0b5c92460c0d5acbfdd1610:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
This comment has been minimized.
This comment has been minimized.
sql/logictest.TestLogic failed with artifacts on master @ 112d0c111c2e9bd54cd13d40ad353cc4b286016e:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
This comment has been minimized.
This comment has been minimized.
sql/logictest.TestLogic failed with artifacts on master @ 852e22a2ad78b1a19bd4f504f7079274c0c95fe5:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ ea6bcca8816a2705e46481eeb7543b16bb7c258c:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ 0604c999494816236a1ed8660405144f071e655b:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ 5989de86b019c154076e42eb0cac1a55ac5c0be3:
Reproduce
To reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
sql/logictest.TestLogic failed with artifacts on master @ e723a85739a2e1b514f5de3221a2177a3a37d0e6:
ReproduceTo reproduce, try: make stressrace TESTS=TestLogic PKG=./pkg/sql/logictest TESTTIMEOUT=5m STRESSFLAGS='-timeout 5m' 2>&1 Parameters in this failure:
Same failure on other branches
|
I was able to reproduce something related to the last hit here with
The expected output for the query is
It seems that the offsets are about the third column. Guessing is that we're forgetting to call I'm also not sure how/why |
I don't see how we could be missing a |
Hm, it looks like the one case where |
68678: opt: don't use outer columns as implicit grouping columns r=DrewKimball a=DrewKimball Previously, the optbuilder logic would add any outer column that is referenced in a grouping context to the set of grouping columns. This is correct in some cases, because outer columns are effectively constant, and can just be removed by norm rules. However, it is incorrect in the case when there are no grouping columns, e.g. a `ScalarGroupBy`. In that case, the `ScalarGroupBy` would inadvertently be converted into a `GroupBy`. This patch modifies the optbuilder to simply ignore outer columns in a grouping context. Fixes #68290 Release note: None 68997: stats: use table descriptors instead of IDs r=postamar a=postamar Previously, the sql stats package inferred various properties about a table from its ID, like if it is a system or a virtual table. However the table descriptor is usually readily or easily available, providing much richer information about a table than its ID. In particular, this allows us to stop collecting stats for views. Release note (sql change): table statistics are no longer collected for views. 69024: colexec: fix bytes corruption for disk-spilled window functions r=DrewKimball a=DrewKimball This patch fixes the `Truncate` method for bytes columns so that it updates the offsets to be non-decreasing up to the new `maxSetLength`. This is necessary in the case when the new `maxSetLength` is greater than the old one. This can happen when a window function has a bytes output column and the `SpillingQueue` that buffers input batches spills to disk. If a batch has trailing nulls up to the last processed index, and it is immediately enqueued to disk, `SpillingQueue` does not call `SetLength` on the batch, so the offsets still need to be updated (in this case by `Truncate`). Fixes #60824 Release note: None 69039: github: route pkg/migration PRs to kv-prs sub-team r=irfansharif a=irfansharif Missed one in #68903. Release note: None Co-authored-by: Drew Kimball <[email protected]> Co-authored-by: Marius Posta <[email protected]> Co-authored-by: irfan sharif <[email protected]>
(sql/logictest).TestLogic failed on master@557346d5fc755db5f64d3e198e47be1f53a4d390:
More
Parameters:
Related:
See this test on roachdash
powered by pkg/cmd/internal/issues
The text was updated successfully, but these errors were encountered: