>> +/* send cancel requests to all connections */ >> +static void >> +cancel_all() >> +{ >> + for (int i = 0; i < nclients; i++) >> + { >> + char errbuf[1]; >> + if (client_states[i].cancel != NULL) >> + (void) PQcancel(client_states[i].cancel, errbuf, >> sizeof(errbuf)); >> + } >> +} >> + >> >> Why in case of errors from PQCancel the error message is neglected? I >> think it's better to print out the error message in case of error. > > Is the message useful for pgbench users? I saw the error is ignored > in pg_dump, for example in bin/pg_dump/parallel.c
I think the situation is different from pg_dump. Unlike pg_dump, if PQcancel does not work, users can fix the problem by using pg_terminate_backend or kill command. In order to make this work, an appropriate error message is essential. Best reagards, -- Tatsuo Ishii SRA OSS LLC English: http://www.sraoss.co.jp/index_en/ Japanese:http://www.sraoss.co.jp