RE: [EXTERNAL] Re: [PATCH] Report the query string that caused a memory error under Valgrind

2023-04-03 Thread Onur Tirtir
uring execution of "select 1;" **13210** Valgrind detected 1 error(s) during execution of "select 1;" Best, Onur -Original Message- From: Tom Lane Sent: Sunday, April 2, 2023 11:14 PM To: Onur Tirtir Cc: peter.eisentr...@enterprisedb.com; pgsql-hackers@lists.postgresql.o

RE: [EXTERNAL] Re: [PATCH] Report the query string that caused a memory error under Valgrind

2023-03-23 Thread Onur Tirtir
Hey Peter, Thank you for reviewing the patch and for your feedback. I believe the v2 patch should be able to handle other protocol messages too. -Original Message- From: Peter Eisentraut Sent: Wednesday, March 22, 2023 7:00 PM To: Onur Tirtir ; pgsql-hackers@lists.postgresql.org

[PATCH] Report the query string that caused a memory error under Valgrind

2023-01-31 Thread Onur Tirtir
We use Valgrind --together with the suppression file provided in Postgres repo-- to test Citus extension against memory errors. We replace /bin/postgres executable with a simple bash script that executes the original postgres executable under Valgrind and then we run our usual regression tests.