HeartSaVioR commented on pull request #31569: URL: https://github.com/apache/spark/pull/31569#issuecomment-780968521
Just a 2 cents, it might be better to convert to `col()` or `$` (I'm now feeling unsafe for this as well though) instead of `Symbol()` on column representations as I feel like they're what we'd prefer as alternatives. Not a requirement as I know you should've done bugging works already. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
