Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adapt all Seq.crossJoin() methods to make use of SeqBuffer #333

Open
tlinkowski opened this issue Mar 2, 2018 · 2 comments
Open

Adapt all Seq.crossJoin() methods to make use of SeqBuffer #333

tlinkowski opened this issue Mar 2, 2018 · 2 comments

Comments

@tlinkowski
Copy link

This is related to #305, where the following Seq methods were declared eligible for reimplementation using SeqBuffer:

As a consequence, the 0.9.13 release has some SeqBuffer-eligible methods that are implemented using SeqBuffer and some that are not.

Expected behavior and actual behavior:

Expected: All Seq.crossJoin() variants are implemented using SeqBuffer.
Actual: All Seq.crossJoin() variants are implements using Seq.toList().

Versions:

  • jOOλ: 0.9.13
@tlinkowski
Copy link
Author

I've noticed that this issue has not been addressed at all (it's not even labeled) so I hereby make a "ping" :)

@lukaseder
Copy link
Member

Thanks a lot for your ping.

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()
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants