On 09.03.21 07:40, Amit Kapila wrote:
Sounds reasonable to me. I also don't see a reason why we need to make this a necessary callback. Some plugin authors might just want 2PC without streaming support.
Sounds okay to me. Probably means we'll have to check for this callback and always skip the prepare for streamed transactions, w/o even triggering filter_prepare, right? (Because the extension requesting not to filter it, but not providing the corresponding callback does not make sense.)
If you're going to together a patch Ajin, I'm happy to review. Best Regards Markus