Comment From: simonbasle
see discussion in gh-26021. cc @mp911de and @jhoeller
Comment From: simonbasle
@mp911de I have updated the PR to using Readable
, but it is not a clear win over the previous version in my opinion due to the fact that I had to instanceof-check the concrete type in order to get the metadata about items (i.e. columns for Row
and out param for OutParameters
). let me know what you think
Comment From: mp911de
As Readable
is not a sealed interface, you're right; there could be potential additional types. In practice, we haven't seen any being around. Also, there was no indication that someone would come up with an additional subtype that isn't a Row or OutParameters.