Github user andrewor14 commented on the pull request:

    https://github.com/apache/spark/pull/7219#issuecomment-119025623
  
    I have some temporary changes here: 
https://github.com/apache/spark/compare/apache:master...andrewor14:fix-maven-build-alternative?expand=1
    
    @vanzin and I spoke offline. Because the new test module actually depended 
on `Logging` from spark-core, we needed to copy the file to the test module to 
avoid introducing a circular dependency. The resulting set of changes actually 
turned out to be quite large, as it changes dependencies across many modules.
    
    I also prefer the simpler fix of introducing a profile that's used only 
internally. Once MSHADE-148 (which is resolved just 3 hours ago, hoorah!) is 
packaged into a new release, we can just upgrade our maven plugin version and 
get rid of the profile. It's a much narrower interface to revert.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to