aokolnychyi commented on PR #50137: URL: https://github.com/apache/spark/pull/50137#issuecomment-2707494998
@cloud-fan, I don't have a strong opinion on this one and can go either way. One point to keep in mind that Spark will not have the guarantee that the newly added information to `TableChange` is going to be respected by the connector. For instance, if we add a new field to `TableInfo`, we won't know if the new field is going to be respected/supported or simply ignored. That said, we can mitigate that via capabilities. I do get the point of not implementing a builder in the connector. -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
