You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm sorry for the delays in these matters. Whether or not an alternative implementation for a variety of jOOλ API should be implemented is a rather time consuming task, time which I currently - very unfortunately - cannot allocate. This doesn't mean that jOOλ will not be improved, but it does mean that the cost / benefit ratio for complex changes like this one is currently not very good. There are lower hanging fruit which have higher chances of making it in the next releases.
YETSDD
added a commit
to YETSDD/jOOL
that referenced
this issue
May 31, 2020
As a consequence, the 0.9.13 release has some SeqBuffer-eligible methods that are implemented using SeqBuffer and some that are not. I change the crossjoin method by implementing it by SeqBuffer() rather than toList()
This is related to #305, where the following
Seq
methods were declared eligible for reimplementation usingSeqBuffer
:crossSelfJoin()
(done in Added SeqBuffer<T> class as a replacement for some of Seq.toList() calls #305)inner(Self)Join()
(done in Added SeqBuffer<T> class as a replacement for some of Seq.toList() calls #305)leftOuter(Self)Join()
(done in Added SeqBuffer<T> class as a replacement for some of Seq.toList() calls #305)crossJoin()
(not done because it needs to be generated usingjOOQ-tools
)As a consequence, the 0.9.13 release has some
SeqBuffer
-eligible methods that are implemented usingSeqBuffer
and some that are not.Expected behavior and actual behavior:
Expected: All
Seq.crossJoin()
variants are implemented usingSeqBuffer
.Actual: All
Seq.crossJoin()
variants are implements usingSeq.toList()
.Versions:
The text was updated successfully, but these errors were encountered: