On Wed, Nov 10, 2021 at 9:53 AM Tom Lane <t...@sss.pgh.pa.us> wrote: > Yeah, that last was pretty much my reaction. I don't know enough about > Perl to be sure how much an unused import costs, but I suspect you're > right that it won't be measurable in context, considering that most of > these test scripts run at least one initdb.
I think focusing on the runtime might be missing the point to some extent. Removing used #include directives from C source files is generally good practice just for cleanliness, and this is the same kind of thing. That said, the argument that future changes are likely to just put the same 'use' directive back seems to me to have some merit. But we've probably spent more energy debating this than the topic deserves. It wouldn't hurt to remove these, and it also won't hurt if we don't. -- Robert Haas EDB: http://www.enterprisedb.com
