sarutak commented on pull request #31569: URL: https://github.com/apache/spark/pull/31569#issuecomment-783006018
> I guess majority of changes are replacing ' being used to column name which I think it wouldn't hurt if we simply change it to $"", but in other cases it might worth to look into per case. Actually I've tried and I found almost all the `'` can be replaced with `$""` but ones in the following tests cannot be done. * EncoderResolutionSuite.scala * ComplexTypeSuite.scala * ObjectExpressionsSuite.scala * NestedColumnAliasingSuite.scala * ReplaceNullWithFalseInPredicateSuite.scala * SimplifyCastsSuite.scala * SimplifyConditionalSuite.scala See #31601 for more details. ---------------------------------------------------------------- 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]
