Hi,
Thank you for your message.
If you are defining all columns in your query and then bypassing the macro at all there is really not performance downside. I would even say that its more efficient as the substitution macro for selected columns doesnt need to resolve (although the gains are really, really small, but definitely not worse.)
Maybe what they meant is that you are getting more columns that you actually need so you can check that out otherwise this seems a correct approach to me.
Kind regards,
Richard Sustek