jzhuge commented on PR #59743: URL: https://github.com/apache/spark/pull/59743#issuecomment-6487010708
@sarutak All three points sound right to me: 1. Agreed: show this key's raw value. 2. Agreed: last-win, with a warning for each dropped default-list occurrence. 3. Keep `spark.plugins` `.distinct` for now; a general mechanism can reconcile them later. Pushed ab19859 for the smaller comments: a pinning test that plain concatenation, first-win, and whole-list de-duplication all fail (plus a LogAppender check for the warning), the two doc wording fixes, and the warning now built from both config keys, ending with "...it is applied where it appears in spark.sql.extensions." -- 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]
