naveenp2708 commented on PR #57644: URL: https://github.com/apache/spark/pull/57644#issuecomment-6401216169
Went back through the rest of the review notes, here’s where each landed: - Nested-array validation (the array(array(...)) case): fixed in 58059a5. - Missing tests for the reconcile paths - COMMENT survival, appended nested fields, the array branch: added in 58059a5. - Typo’d or duplicate reserved column being stripped from both sides: handled in 742a80b by validateDeclaredReservedFields. - The elements.scala one (a same-graph reader hitting the internal-error guard before the clean duplicate error): it’s a narrow case and still fails either way, just with a less specific error. Leaving it out of this PR to keep the scope tight and tracking it as a follow-up. -- 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]
