On 2021/03/05 16:33, Michael Paquier wrote:
On Fri, Mar 05, 2021 at 01:30:11PM +0900, Fujii Masao wrote:if ((env = getenv("PGDATABASE")) != NULL && *env != '\0') dbName = env; - else if (login != NULL && *login != '\0') - dbName = login; + else if ((env = getenv("PGUSER")) != NULL && *env != '\0') + dbName = env; else - dbName = ""; + dbName = get_user_name_or_exit(progname); If dbName is set by libpq, the above also is not necessary?If you remove this part, pgbench loses some log information if PQconnectdbParams() returns NULL, like on OOM if the database name is NULL. Perhaps that's not worth caring about here for a single log failure, but that's the reason is why I left this part around.
Understood. Thanks! Regards, -- Fujii Masao Advanced Computing Technology Center Research and Development Headquarters NTT DATA CORPORATION
