Thanks yes then I agree, we shouldn’t log the parameters. Looks like this was introduced as https://hibernate.atlassian.net/browse/HHH-12878 .

I don’t think this should be based on the show_sql property though, that’s meant for a different aspect and it would be confusing to have it interact with this aspect.

I actually think we should never log parameters in the stacktrace of a {{StaleStateException}}so we could keep it simple and avoid any configuration property?