Github user maropu commented on the pull request:

    https://github.com/apache/spark/pull/4176#issuecomment-72786951
  
    I wonder if stopping the process is the best solution.
    If there is only one illegal entry in a last line, we need to re-try 
loading a whole file,
    which is time-consuming.
    
    An other idea is that illegal entries are silently re-directed into a file 
or something.
    Finally, # of the re-directed entries is only output in the last phase of 
bulk-loading.
    Moreover, I think that it'd be better to add a new API to append these 
entries into existing Graph
    such as GraphOps.addEdges(val edges: RDD[Edge[ED]]). 
     


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