forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit5935092
committed
Fix poorly written regression test
bd10ec5 added code to allow redundant functionally dependent GROUP BYcolumns to be removed using unique indexes and NOT NULL constraints asproofs of functional dependency. In that commit, I (David) added a testto ensure that when there are multiple indexes available to remove columnsthat we pick the index that allows us to remove the most columns. Thistest was faulty as it assumed the t3 table's primary key index was validto use as functional dependency proof, but that's not the case sincethat's defined as deferrable.Here we adjust the tests added by that commit to use the t2 table instead.That's defined with a non-deferrable primary key.Author: songjinzhou <tsinghualucky912@foxmail.com>Author: David Rowley <dgrowleyml@gmail.com>Reviewed-by: Japin Li <japinli@hotmail.com>Discussion:https://postgr.es/m/tencent_CD414C79D39668455DF80D35143B87634C08@qq.com1 parent217919d commit5935092
2 files changed
+38
-38
lines changedLines changed: 25 additions & 25 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1448 | 1448 |
| |
1449 | 1449 |
| |
1450 | 1450 |
| |
1451 |
| - | |
| 1451 | + | |
1452 | 1452 |
| |
1453 | 1453 |
| |
1454 |
| - | |
| 1454 | + | |
1455 | 1455 |
| |
1456 | 1456 |
| |
1457 | 1457 |
| |
1458 |
| - | |
1459 |
| - | |
| 1458 | + | |
| 1459 | + | |
1460 | 1460 |
| |
1461 | 1461 |
| |
1462 | 1462 |
| |
1463 |
| - | |
1464 |
| - | |
| 1463 | + | |
| 1464 | + | |
1465 | 1465 |
| |
1466 | 1466 |
| |
1467 | 1467 |
| |
1468 |
| - | |
1469 |
| - | |
| 1468 | + | |
| 1469 | + | |
1470 | 1470 |
| |
1471 | 1471 |
| |
1472 | 1472 |
| |
1473 | 1473 |
| |
1474 | 1474 |
| |
1475 |
| - | |
| 1475 | + | |
1476 | 1476 |
| |
1477 | 1477 |
| |
1478 | 1478 |
| |
1479 |
| - | |
1480 |
| - | |
| 1479 | + | |
| 1480 | + | |
1481 | 1481 |
| |
1482 | 1482 |
| |
1483 | 1483 |
| |
1484 | 1484 |
| |
1485 |
| - | |
| 1485 | + | |
1486 | 1486 |
| |
1487 | 1487 |
| |
1488 | 1488 |
| |
1489 |
| - | |
1490 |
| - | |
| 1489 | + | |
| 1490 | + | |
1491 | 1491 |
| |
1492 | 1492 |
| |
1493 | 1493 |
| |
1494 |
| - | |
1495 |
| - | |
1496 |
| - | |
| 1494 | + | |
| 1495 | + | |
| 1496 | + | |
1497 | 1497 |
| |
1498 | 1498 |
| |
1499 | 1499 |
| |
1500 |
| - | |
1501 |
| - | |
| 1500 | + | |
| 1501 | + | |
1502 | 1502 |
| |
1503 | 1503 |
| |
1504 | 1504 |
| |
1505 | 1505 |
| |
1506 | 1506 |
| |
1507 |
| - | |
1508 |
| - | |
1509 |
| - | |
1510 |
| - | |
| 1507 | + | |
| 1508 | + | |
| 1509 | + | |
| 1510 | + | |
1511 | 1511 |
| |
1512 | 1512 |
| |
1513 | 1513 |
| |
1514 |
| - | |
1515 |
| - | |
| 1514 | + | |
| 1515 | + | |
1516 | 1516 |
| |
1517 | 1517 |
| |
1518 | 1518 |
| |
|
Lines changed: 13 additions & 13 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
507 | 507 |
| |
508 | 508 |
| |
509 | 509 |
| |
510 |
| - | |
| 510 | + | |
511 | 511 |
| |
512 | 512 |
| |
513 | 513 |
| |
514 |
| - | |
| 514 | + | |
515 | 515 |
| |
516 | 516 |
| |
517 |
| - | |
518 |
| - | |
| 517 | + | |
| 518 | + | |
519 | 519 |
| |
520 | 520 |
| |
521 | 521 |
| |
522 | 522 |
| |
523 |
| - | |
| 523 | + | |
524 | 524 |
| |
525 | 525 |
| |
526 | 526 |
| |
527 |
| - | |
| 527 | + | |
528 | 528 |
| |
529 | 529 |
| |
530 |
| - | |
531 |
| - | |
532 |
| - | |
| 530 | + | |
| 531 | + | |
| 532 | + | |
533 | 533 |
| |
534 | 534 |
| |
535 | 535 |
| |
536 | 536 |
| |
537 |
| - | |
538 |
| - | |
539 |
| - | |
540 |
| - | |
| 537 | + | |
| 538 | + | |
| 539 | + | |
| 540 | + | |
541 | 541 |
| |
542 | 542 |
| |
543 | 543 |
| |
|
0 commit comments
Comments
(0)