Repository navigation
Commit 631b53d
Keep concurrent commits on rewrite retry and check the rewrite keeps every active file
A rewrite that raced an append failed its retry on the serializable
added-files check, even though the retry replans against the new head.
Skip that check, as Java does, and instead refuse a rewrite whose new
manifests do not hold the same number of active files as the ones they
replace.
Co-authored-by: Isaac <[email protected]>1 parent cadafef commit 631b53d
2 files changed
Lines changed: 63 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1314 | 1314 | | |
1315 | 1315 | | |
1316 | 1316 | | |
| 1317 | + | |
| 1318 | + | |
| 1319 | + | |
| 1320 | + | |
| 1321 | + | |
| 1322 | + | |
| 1323 | + | |
| 1324 | + | |
| 1325 | + | |
1317 | 1326 | | |
1318 | 1327 | | |
1319 | 1328 | | |
| |||
1370 | 1379 | | |
1371 | 1380 | | |
1372 | 1381 | | |
| 1382 | + | |
| 1383 | + | |
| 1384 | + | |
1373 | 1385 | | |
1374 | 1386 | | |
1375 | 1387 | | |
| |||
1419 | 1431 | | |
1420 | 1432 | | |
1421 | 1433 | | |
| 1434 | + | |
1422 | 1435 | | |
1423 | 1436 | | |
1424 | 1437 | | |
| |||
1438 | 1451 | | |
1439 | 1452 | | |
1440 | 1453 | | |
| 1454 | + | |
1441 | 1455 | | |
1442 | 1456 | | |
1443 | 1457 | | |
| 1458 | + | |
| 1459 | + | |
| 1460 | + | |
| 1461 | + | |
| 1462 | + | |
| 1463 | + | |
| 1464 | + | |
| 1465 | + | |
1444 | 1466 | | |
1445 | 1467 | | |
1446 | 1468 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
15 | 15 | | |
16 | 16 | | |
17 | 17 | | |
| 18 | + | |
18 | 19 | | |
19 | 20 | | |
20 | 21 | | |
21 | 22 | | |
22 | 23 | | |
23 | 24 | | |
24 | | - | |
| 25 | + | |
| 26 | + | |
25 | 27 | | |
26 | 28 | | |
27 | 29 | | |
| |||
303 | 305 | | |
304 | 306 | | |
305 | 307 | | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
| 322 | + | |
| 323 | + | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
| 336 | + | |
| 337 | + | |
| 338 | + | |
| 339 | + | |
| 340 | + | |
| 341 | + | |
| 342 | + | |
| 343 | + | |
| 344 | + | |
| 345 | + | |
0 commit comments