Vladimir Sitnikov
Vladimir Sitnikov
Recent PostgreSQL would report `search_path` change to the clients, see https://github.com/postgres/postgres/commit/28a1121fd91235e09793c9ce64cb5539719afa30 The driver would be able to adjust/invalidate the cache in that case. You could ask PostgreSQL developers to backport...
>So you imply that without the GUC_REPORT patch , and with setting search_path wrapped inside a function, then we would get jeopardized results ? I think so.
@pochopsp, have you tried using the default `org.apache.jmeter.testelement.TestElementSchema#getEnabled` for the attributes? Is there a need for a separate property?
Looks like the regression tests pass, however, we should add new tests for the disabled arguments.
"nullability" is not trivial, so it might take some time to properly annotate and identify nullable types. The general advice is assume "everything is not nullable by default" (==almost never...
In my experience, checkerframework nullability verification takes time, so I tend to activate it only when `-PenableCheckerframework` parameter is specified. Then checkerframework executes in a single CI job only. See...
I would suggest merging `errorprone` first as it would require much fewer changes.
One more thing: `List` is a good enough replacement for `List
@marcphilipp , sorry for asking the same thing twice, however, could you please clarify why do you mean `Arguments.from(List)`, `Arguments argumentsFrom(List)`, and `ArgumentSet argumentSetFrom(String name, List)` relate to "from existing...
Could you elaborate on the reasons for going against Java specification?