streams-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mfranklin <...@git.apache.org>
Subject [GitHub] incubator-streams pull request: added support for file: and hdfs: ...
Date Wed, 24 Sep 2014 17:53:38 GMT
Github user mfranklin commented on the pull request:

    https://github.com/apache/incubator-streams/pull/85#issuecomment-56711278
  
    IMO, the community should be organizing code to isolate difficult to test dependencies
and creating issues in the tracker to do proper integration testing across external systems.
 
    
    In the case of this pull request, the getURI method should be testable without having
to spin up any external dependencies on HDFS.  If we can cover those types of methods with
good tests and ensure HDFS is isolated as much as possible, it would meet the minimum Ryan
proposed on the list.


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

Mime
View raw message