HeartSaVioR edited a comment on pull request #29113: URL: https://github.com/apache/spark/pull/29113#issuecomment-662106547
It might be only me, but even we are trying to guarantee backward/forward compatibility with event logs, I wouldn't think it's for covering three major versions, even not latest one for 1.x. IMHO the config is not needed and we can just remove the compatibility, but again that's only me, and it'd be nice to initiate discussion thread in dev@ list to reach consensus though. Could you go through the dev@ mailing list to ask how we deal with this? (I'll post my opinion separately so you only need to include your change.) I guess the discussion thread won't hold too long, as it might be just obvious. ---------------------------------------------------------------- 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]
