search

arenadata / gpdb

Arenadata DB
https://docs.arenadata.io/en/ADB/current/introduction/intro.html
Apache License 2.0
40 stars 22 forks source link

Mark CurrentMemoryContext as volatile #1070

Closed dkovalev1 closed 1 week ago

dkovalev1 commented 3 weeks ago

Statement cancellation handler works in it’s own thread and perform memory context switches. Main worker thread in error handling code also uses memory contexts. Memory context switch is performed via global variable CurrentMemoryContext which is not thread safe.

This patch marks CurrentMemoryContext as volatile to make it thread safe.

BenderArenadata commented 3 weeks ago

Allure report https://allure.adsw.io/launch/82192

dkovalev1 commented 1 week ago

The task was closed, this change is not relevant, so closing the PR.