naveenp2708 commented on PR #57644: URL: https://github.com/apache/spark/pull/57644#issuecomment-5489082422
@szehon-ho good catch, fixed. appendEngineOwnedReservedFields now keeps the declared reserved column's spelling and position and swaps in only the engine type, so the read path matches an existing target that has it in a different casing. Added the regression test you described and confirmed it fails without the change. Honest caveat: this covers declaring the column, not omitting it on an existing upper-cased target (the read path is analysis-time, so it can't see the catalog spelling). Can file a follow-up for that if you'd like. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
