Aleksander Alekseev <[email protected]> writes: Hello!
> Hi, > >> So what I actually need to do is *remove* part which executes `if >> (IsA(plan, JsonTablePathScan))` branch since now it is executed >> without the patch. I will submit an updated version. > > Here is the simplified patch. I've reviewed the new version and now the coverage with this version is 92%, the version 2 was 95%, probably this is because the test doesn't cover the "UNION" case. I still wonder why you don't want to cover the 100% if it's just a small changes in the test, but for now, this does third version it does increase the coverage by 42%. Regards! -- Jonathan Gonzalez V. EDB https://www.enterprisedb.com
