Commit 407771f
committed
roachtest: increase server.sqlliveness.ttl in copyfrom
`copyfrom/atomic` often fails with `RETRY_COMMIT_DEADLINE_EXCEEDED`
error. Based on some guidance from KV folks, it sounds like (apart from
the closed TS system that has already been adjusted) the most likely
explanation is that the lease on a descriptor has expired while the txn
was running. Before 24.1 we used expiry based leasing, then we
introduced session based leasing, and the migration to only use that has
been completed in the beginning of the year. If I'm reading the code
right, then the lease duration depends on the SQL liveness TTL, so this
commit bumps the relevant cluster setting from 40s to 5m to give enough
time for atomic COPY to finish.
I did 50 runs of the test, and all of them passed.
Release note: None1 parent c619ca9 commit 407771f
1 file changed
+8
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
150 | 150 | | |
151 | 151 | | |
152 | 152 | | |
153 | | - | |
154 | | - | |
155 | | - | |
156 | | - | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
157 | 161 | | |
158 | 162 | | |
159 | 163 | | |
| |||
0 commit comments