forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit38ff135
committed
Cleanup some code related to pgbench log checks in TAP tests
This fixes a couple of problems within the so-said code of this commitsubject:- Replace the use of open() with slurp_file(), fixing an issue reportedby buildfarm member fairywren whose perl installation keep around CRLFcharacters, causing the matching patterns for the logs to fail.- Remove the eval block, which is not really necessary.This set of issues has come into light after fixing a different issuewithc13585f, and this is wrong since this code has been introduced.Reported-by: Andrew Dunstan, and buildfarm member fairywrenAuthor: Michael PaquierReviewed-by: Andrew DunstanDiscussion:https://postgr.es/m/0f49303e-7784-b3ee-200b-cbf67be2eb9e@dunslane.netBackpatch-through: 111 parent3af1094 commit38ff135
1 file changed
+18
-19
lines changedLines changed: 18 additions & 19 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1193 | 1193 |
| |
1194 | 1194 |
| |
1195 | 1195 |
| |
1196 |
| - | |
1197 |
| - | |
1198 |
| - | |
1199 |
| - | |
1200 |
| - | |
1201 |
| - | |
1202 |
| - | |
1203 |
| - | |
1204 |
| - | |
1205 |
| - | |
1206 |
| - | |
| 1196 | + | |
| 1197 | + | |
| 1198 | + | |
| 1199 | + | |
| 1200 | + | |
| 1201 | + | |
| 1202 | + | |
| 1203 | + | |
| 1204 | + | |
| 1205 | + | |
| 1206 | + | |
| 1207 | + | |
| 1208 | + | |
| 1209 | + | |
| 1210 | + | |
1207 | 1211 |
| |
1208 |
| - | |
1209 |
| - | |
1210 |
| - | |
1211 |
| - | |
1212 |
| - | |
| 1212 | + | |
| 1213 | + | |
1213 | 1214 |
| |
1214 |
| - | |
1215 |
| - | |
| 1215 | + | |
1216 | 1216 |
| |
1217 |
| - | |
1218 | 1217 |
| |
1219 | 1218 |
| |
1220 | 1219 |
| |
|
0 commit comments
Comments
(0)