You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/03/06 15:48:50 UTC
[GitHub] [spark] srowen commented on issue #27808:
[SPARK-30994][BUILD][FOLLOW-UP] Change scope of xml-apis to include it and
add xerces in SBT as dependency override
srowen commented on issue #27808: [SPARK-30994][BUILD][FOLLOW-UP] Change scope of xml-apis to include it and add xerces in SBT as dependency override
URL: https://github.com/apache/spark/pull/27808#issuecomment-595829043
Thanks everyone! on the one hand I think not catching it was my original PR's error, but getting that right wouldn't have surfaced this in tests, so thanks to @tgravescs for catching the issue downstream. Thanks @HyukjinKwon for the fix. I don't think we'll attempt this in 2.x unless there's demand.
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org