Fixes gh-23962
Comment From: virtual-machinist
@maheksaluja , any ETA on when this PR is to be merged?
Comment From: kazuki43zoo
I would be happy if merging.
Comment From: aegliv
I would also love to see this PR merged
Comment From: snicoll
Thanks for the PR but we're not keen on having a composite that merges the parameter names behind the scenes. The same can be accomplished by MapSqlParameterSource, it even has a addValues for such a scenario. Please consider using that, and we can continue the discussion if it turns out additional changes might be required.
Comment From: virtual-machinist
@snicoll , can you elaborate on how is MapSqlParameterSource#addValues a replacement for multiple SqlParameterSource-s that are not backed by a map?
The way I see it the proposed PR makes no assumptions about the parameter sources used - one can be backed by a map, the other by a bean using reflection etc.
Comment From: snicoll
I understand that, but it's the merging of properties that we're not keen on. It can have subtle effect and increase the maintenance cost on our end. However, applying the merge in your local setup if that makes sense is already supported is what I was trying to say.