Github user JoshRosen commented on the pull request:

    https://github.com/apache/spark/pull/5694#issuecomment-111736875
  
    After some reflection, I think that the suggestions outlined in [my last 
high-level 
comment](https://github.com/apache/spark/pull/5694#issuecomment-111625224) 
might take a while to implement since they're essentially suggesting a 
different high-level organization of some of the test-selection logic. Although 
I think in the long-term we may want a design closer to what I proposed there, 
for now I think we should set that aside and just focus on making the smaller 
changes that I suggested to get the current design working. If we do that, I'll 
commit this and will submit a followup PR to explore my reorganization ideas.
    
    I'll go take a look at my previous comments now and will flag anything that 
doesn't need to be done as part of this PR.


---
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