[ 
https://issues.apache.org/jira/browse/FLINK-2785?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14945261#comment-14945261
 ] 

ASF GitHub Bot commented on FLINK-2785:
---------------------------------------

Github user fhueske commented on a diff in the pull request:

    https://github.com/apache/flink/pull/1205#discussion_r41284938
  
    --- Diff: 
flink-staging/flink-gelly-scala/src/main/scala/org/apache/flink/graph/scala/Graph.scala
 ---
    @@ -126,6 +126,122 @@ object Graph {
         wrapGraph(jg.Graph.fromTupleDataSet[K, VV, EV](javaTupleEdges, mapper, 
env.getJavaEnv))
       }
     
    +  /**
    +  * Creates a Graph with from a CSV file of vertices and a CSV file of 
edges
    +  * 
    +  * @param pathVertices The file path containing the vertices.
    +  * @param vertexValue Defines whether the vertices have associated values.
    +  * If set to false, the vertex input is ignored and vertices are created 
from the edges file.
    +  * True by default.
    +  * @param lineDelimiterVertices The string that separates lines in the 
vertices file.
    +  * It defaults to newline.
    +  * @param fieldDelimiterVertices The string that separates vertex Ids 
from vertex values
    +  * in the vertices file.
    +  * @param quoteCharacterVertices The character to use for quoted String 
parsing
    +  * in the vertices file. Disabled by default.
    +  * @param ignoreFirstLineVertices Whether the first line in the vertices 
file should be ignored.
    +  * @param ignoreCommentsVertices Lines that start with the given String 
in the vertices file
    +  * are ignored, disabled by default.
    +  * @param lenientVertices Whether the parser should silently ignore 
malformed lines in the
    +  * vertices file.
    +  * @param includedFieldsVertices The fields in the vertices file that 
should be read.
    +  * By default all fields are read.
    +  * @param pathEdges The file path containing the edges.
    +  * @param edgeValue Defines whether the edges have associated values. 
True by default.
    --- End diff --
    
    `hasEdgeValues`?


> Implement Graph's fromCsvReader in Gelly-Scala
> ----------------------------------------------
>
>                 Key: FLINK-2785
>                 URL: https://issues.apache.org/jira/browse/FLINK-2785
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Gelly
>            Reporter: Vasia Kalavri
>            Assignee: Vasia Kalavri
>            Priority: Minor
>
> Graphs's {{fromCsvReader}} method is currently missing from the Gelly Scala 
> API. It cannot be implemented as a simple wrapper over the Java method, 
> because the Java method returns a {{GraphCsvReader}}, which in return creates 
> a Graph after specifying types with appropriate methods. The Scala version of 
> the method can be more nicely implemented using the {{ScalaCsvInputFormat}}.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to