Github user okram commented on the pull request:

    
https://github.com/apache/incubator-tinkerpop/pull/299#issuecomment-215977029
  
    I don't think you need to put as may interrupt checks. Most steps inherit 
from `AbstractStep` and thus, if `AbstractStep.next()` and 
`AbstractStep.hasNext()` have the interrupt check, then all the 
`processNextStart()` in subclasses don't need to check for the interrupt. This 
should greatly reduce the amount of classes touched and requirements on 
providers with their own step implementations. In short, they just need to make 
sure they extend `AbstractStep`. However, with that said, I still think you 
need the `VertexProgramStep` work in there as you have it.


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

Reply via email to