On Thursday, May 22, 2014, Szymon <notifications@github.com> wrote:

> In
> cassovary-core/src/main/scala/com/twitter/cassovary/util/io/ListOfEdgesGraphReader.scala:
>
> >   * Does not check for duplicate edges or nodes.
> >   *
> >   * You can optionally specify which files in a directory to read. For example, you may have files starting with
> >   * "part-" that you'd like to read. Only these will be read in if you specify that as the file prefix.
> >   *
> > - * In each file, a directed edges is defined by a pair of integers: from and to.
> > - * For example,
> > + * You should also specify `nodeNumberer`, `idReader`, `separator` between nodes in an edge
> > + * and `quotationMark` if used with node ids.
> > + *
> > + * For a default version for `Int` graphs see [[ListOfEdgesGraphReader.forIntIds]] builder method.
> > + *
> > + * In each file, a directed edges is defined by a pair of T: from and to.
> > + * For example, we use `String` ids with `"` `quotationMark` and ` ` (space) `separator`, when
>
> That is needed if we want node ids to have space or other special char.
>


It is a bit cumbersome to have " around every node id. Perhaps choose a
default single char separator (space) and allow it to be overridden

> —
> Reply to this email directly or view it on GitHub<https://github.com/twitter/cassovary/pull/71/files#r12963341>
> .
>