On 13/06/2013 3:33 PM, Marek Potociar wrote:
> Marek,
>
> I hope you didn't misinterpret the above post as FUD because that is not my intent. I have contributed to Jersey for many years in the form of bug reports and patches. I genuinely want to work with you to improve the product. Now that Jersey 2.0 is officially out I'd like to get your feedback on how we can integrate the patches I posted on June 2011 at https://java.net/jira/browse/JERSEY-710
>
> I am more than willing to update these patches to Jersey 2.0 but I'd like your guidance on the kind of changes you're looking for. Would you accept these patches as-is (once they're updated to Jersey 2.0) or do you have other concerns to be addressed?
>
> Thank you,
> Gili
> I think we should start by you opening a new github pull request for jersey for those patches (assuming you have signed OCA, which, if you contributed in the past, hopefully have). Once these commits pass our verification build, we will review them and either accept them "as-is" or add our review comments to the pull request asking you to incorporate those in the code. Once the commits are verified, reviewed and accepted, we will eventually merge the pull request.
>
> I shall also mention that for any new major feature contributions it is now required that such contributions are accompanied with a dedicated chapter in our user guide and an example.
>
> As for the types of contributions, we welcome wide range of patches:
> - bug fixes, new tests
> - documentation updates, translations
> - standalone examples
> - improvements (performance, configuration options, user friendlier APIs...)
> - extensions & new standalone features (see the note on accompanying docs and example above), e.g.:
> - integration with other containers, client transports, injection frameworks, templating engines...
> - support for additional media types
> - bindings for other JVM languages
>
> I'm looking forward to your contributions,
> Marek
>
> P.S. When I referred to FUD, I meant the general discussions whether Jersey 2.0 is out or not.
>
Okay, that's good news. I reviewed the
https://java.net/jira/browse/JERSEY-710 patch and the only breaking
changes I spotted were limited to internal methods not normally
accessible to end-users. I will update the patch for Jersey 2.x and
create a pull request as you suggested.
Thank you,
Gili