Github user vanzin commented on the issue:

    https://github.com/apache/spark/pull/16311
  
    >me: if you make a SQL change, that will trigger the exclusion of 
"ExtendedYarnTest"
    >Can you elaborate on why the second part of this follows from the first?
    
    Sorry, I think that's not accurate, actually. I think it works more like 
this: if you make a change, but you don't change any files under the YARN 
module, then some slower YARN tests are skipped by excluding that tag. I don't 
remember exactly what the code does, so it might add the exclusion in more 
cases, but that is the idea at least.
    
    You can try to run the tests with the argument to exclude the tags locally 
to see if that works, but I really do not remember the command.
    
    > Should spark-tags not inherit from the root POM?
    
    It would require some duplication (e.g. version of shared dependencies) but 
is probably doable. Don't know if there are any other implications.


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

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

Reply via email to