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

Java DSL cannot create streaming marshaller / unmarshaller #35

Closed
akka-ci opened this issue Sep 8, 2016 · 7 comments
Closed

Java DSL cannot create streaming marshaller / unmarshaller #35

akka-ci opened this issue Sep 8, 2016 · 7 comments
Assignees
Labels
1 - triaged Tickets that are safe to pick up for contributing in terms of likeliness of being accepted 3 - in progress Someone is working on this ticket t:marshalling
Milestone

Comments

@akka-ci
Copy link

akka-ci commented Sep 8, 2016

Issue by jypma
Thursday Sep 01, 2016 at 13:37 GMT
Originally opened as akka/akka#21340


The Java Marshaller and Unmarshaller classes are missing a few of the scala variants that make it impossible to write streaming marshallers and unmarshallers in Java.

I've already written the needed additions for Marshaller and Unmarshaller.

Shall I create a PR?

@akka-ci akka-ci added this to the 2.4.x milestone Sep 8, 2016
@akka-ci akka-ci added 3 - in progress Someone is working on this ticket t:http labels Sep 8, 2016
@akka-ci
Copy link
Author

akka-ci commented Sep 8, 2016

Comment by ktoso
Thursday Sep 01, 2016 at 13:56 GMT


Hm, what specifically do you mean saying "streaming marshaller / unmarshaller"?
But yeah, please PR! :)

@akka-ci
Copy link
Author

akka-ci commented Sep 8, 2016

Comment by jypma
Thursday Sep 01, 2016 at 14:08 GMT


PR created. Turns out the "streaming marshaller" can in fact be written in plain Java. That leaves just the unmarshaller side, plus a few random other fixes.

@akka-ci
Copy link
Author

akka-ci commented Sep 8, 2016

Comment by jypma
Monday Sep 05, 2016 at 08:35 GMT


@ktoso tests passed on the PR (akka/akka#21341). How do you feel about the additions?

@akka-ci
Copy link
Author

akka-ci commented Sep 8, 2016

Comment by ktoso
Monday Sep 05, 2016 at 08:37 GMT


Had a crazy week, back to reviewing.

@ktoso ktoso added 1 - triaged Tickets that are safe to pick up for contributing in terms of likeliness of being accepted and removed t:http labels Sep 8, 2016
@ktoso ktoso removed this from the 2.4.x milestone Sep 12, 2016
@jlprat
Copy link
Member

jlprat commented Feb 8, 2017

Shouldn't this be closed as the PR referenced here is long time closed?

@ktoso
Copy link
Member

ktoso commented Feb 8, 2017

Seems it did not make it over to Akka-HTTP during repo change: akka/akka#21341 (comment)

@jypma are you still interested in making this happen?

@jrudolph jrudolph added this to the duplicate milestone Feb 19, 2018
@jrudolph
Copy link
Member

Afaics this has been fixed by now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 - triaged Tickets that are safe to pick up for contributing in terms of likeliness of being accepted 3 - in progress Someone is working on this ticket t:marshalling
Projects
None yet
Development

No branches or pull requests

5 participants