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 2022/03/17 09:18:58 UTC

[GitHub] [spark] alkis edited a comment on pull request #35874: [SPARK-38569][BUILD] Rename `external` to `vendor` top level dir

alkis edited a comment on pull request #35874:
URL: https://github.com/apache/spark/pull/35874#issuecomment-1070589575


   > Although `Apache Spark` is irrelevant to `bazel`, I fully understand the requirement from downstream projects here. What we need is simply an officially discussion with a broader community audience.
   
   Thank you Dongjoon. This is my first contribution to `Apache Spark` so bear with my perhaps silly questions. Could you advice what topics we want to discuss - the move itself, the final directory name or something else? While the PR looks big, at its core this is a single directory rename (maybe there are risks I am unaware of?).


-- 
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-unsubscribe@spark.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org