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

Adding value extractors for primitive iterable types #374

Closed
wants to merge 1 commit into from

Conversation

marko-bekhta
Copy link

Hi all,

as per http://beanvalidation.org/2.0/spec/#valueextractordefinition Bean validation 2.0 offers so called ValueExtractor mechanism to allow adding constraints for containers. I've added a couple of such extractors in a separate module for primitive iterables.
Do you think this would be a good addition for Eclipse Collections? If so - it'll be great if you can point me to other collections that are not yet included but should probably have such extractors. Looking forward for the feedback. Thanks!

@nikhilnanivadekar
Copy link
Contributor

@marko-bekhta firstly apologies for the delay in reply.
I like the idea of having value extractors, but I would not want to have it in the same repository. We can have more than 1 repository for a project. Few reasons why:

  1. Addition of new dependencies: I want to keep Eclipse Collections dependency free. Also, to add new dependencies we will have to carefully look at the licenses and get it reviewed by Eclipse Foundation
  2. Release process: I would like to keep the release schedule of value extractors and Eclipse Collections separate. I expect the value extractor project to be fairly agile at start and then less aggressive when it stabilizes.
  3. Ease of experimentation: Eclipse Collections is a stable and mature project, where as I can see Value Extractors to be experimental at start and then mature over time. I am fine with that, but would like to break it up in to separate parts.
  4. Copyrights: From your PR the copyrights need to be updated. Please refer to: http://www.eclipse.org/projects/handbook/#ip-copyright-headers

How soon do you need value extractors? This will help me prioritize the conversations for new repository creation, etc.

@marko-bekhta
Copy link
Author

Hi @nikhilnanivadekar no worries, it's fine :)
You have good points on having it separate. I was actually thinking that adding a new module to the current project would allow to package a separate releasable jar. But it would mean that both main collection jar as well as a value extractors should be released at the same time. So having a different project sounds better.

Copyrights: From your PR the copyrights need to be updated. Please refer to: http://www.eclipse.org/projects/handbook/#ip-copyright-headers

Hah, haven't seen that one. I'll make the changes and force-push the commit.

As for the last question - there's no rush on my side, so whenever you have time for this, is fine for me.

Also if I missed some collections - let me know, I can add extractors for those as well (either now or when the new repo is ready, depending on what works better for you)

Signed-off-by: marko-bekhta <marko.prykladna@gmail.com>
@marko-bekhta
Copy link
Author

Hi @nikhilnanivadekar any news on this? I've fixed the license headers and in addition rebased on top of lates master changes.

@nikhilnanivadekar
Copy link
Contributor

Apologies @marko-bekhta I forgot about this. I will follow up on getting a different repository for value extractors and post you. Thanks for your patience!

@marko-bekhta
Copy link
Author

hi @nikhilnanivadekar :) how's the follow up on a repository? Maybe I can help with something there ?

@nikhilnanivadekar
Copy link
Contributor

I am pretty tied up at the moment. @itohiro73 can you please help take this forward?

@marko-bekhta
Copy link
Author

Hi, any news? If you can point me into the right direction I can try to get this moving forward. Thanks!

@donraab
Copy link
Contributor

donraab commented Jul 12, 2020

@nikhilnanivadekar @marko-bekhta this PR is almost two years old. Should we revisit, rethink or close this?

@marko-bekhta
Copy link
Author

@donraab that's a good question 😉 I haven't seen much interest/need in these particular extractors since the time this PR was opened. Hence I'll leave it up to you to decide how you'd like to move forward with it. If any help is needed from me to bring this work up to date - just let me know.

@nikhilnanivadekar
Copy link
Contributor

Apologies for the tardy reply. We have 2 options:

  1. Create a new module in Eclipse Collections repo so that value extractors can be released as a part of the main project
  2. Create a separate repo for value extractors

Both the approaches have their pros and cons.

  • If we add to the existing repo, we essentially add new dependencies which will need to go through Change Request approval, etc. Also, I personally prefer to have the main project free of external dependencies

  • If we create a new repo then it adds the additional steps to setup the infrastructure for releases, etc, etc.

Question to @marko-bekhta how do you foresee the extractors being used? I am not an hibernate expert so don't quite know how the value extractors will be situated and used. If you have documentation or code or blogs where I can get myself educated with this, it will help make us the decision.

Appreciate your patience and support!
Cheers!

@donraab donraab closed this Sep 21, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants