Open lintool opened 7 years ago
If we look at the Node API, we have:
def randomOutboundNodeSet(numResults: Int, rnd: Random): Seq[Int]: https://github.com/twitter/cassovary/blob/master/cassovary-core/src/main/scala/com/twitter/cassovary/graph/Node.scala#L123
def randomOutboundNodeSet(numResults: Int, rnd: Random): Seq[Int]
def outboundNodes(): CSeq[Int]: https://github.com/twitter/cassovary/blob/master/cassovary-core/src/main/scala/com/twitter/cassovary/graph/Node.scala#L89
def outboundNodes(): CSeq[Int]
Note that one returns a CSeq and the other returns a Seq. Should this be made consistent?
CSeq
Seq
Good catch. Will fix.
pankajgupta
commented
7 years ago pankajgupta
commented
7 years ago
If we look at the Node API, we have:
def randomOutboundNodeSet(numResults: Int, rnd: Random): Seq[Int]: https://github.com/twitter/cassovary/blob/master/cassovary-core/src/main/scala/com/twitter/cassovary/graph/Node.scala#L123def outboundNodes(): CSeq[Int]: https://github.com/twitter/cassovary/blob/master/cassovary-core/src/main/scala/com/twitter/cassovary/graph/Node.scala#L89Note that one returns a
CSeqand the other returns aSeq. Should this be made consistent?