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
+htp #18837 JSON framing and framed entity streaming directives #20778
+htp #18837 JSON framing and framed entity streaming directives #20778
Conversation
Test FAILed. |
} | ||
|
||
/** | ||
* Simple rendering mode, similar to [[Compact]] however interspersing elements with a `\n` character. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
\n character?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
\n
makes it nicely formatted though
LGTM |
Test FAILed. |
Test FAILed. |
39d6b17
to
d2e47c0
Compare
Test PASSed. |
This is finally complete, including javadsl and documentation (after oh-so-many-years). |
Test FAILed. |
Very weird timeout... I blame jenkins :/ |
Test FAILed. |
* This is achieved by applying the implicitly provided (in the following order): | ||
* | ||
* - 1st: [[FramingFlow]] in order to chunk-up the incoming [[ByteString]]s according to the | ||
* `Content-Type` aware framing (for example, [[common.EntityStreamingSupport.bracketCountingJsonFraming]]). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bracketCountingJsonFraming
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hah, will fix :)
Test FAILed. |
added CSV examples updated docs EntityStreamingSupport is now an entry point, to all streaming things both read and write side use it it's easy to extend as well
5e50bde
to
da23f6d
Compare
da23f6d
to
8a1f8f2
Compare
The only failures were in a specific JsonFraming spec - those were legitimate bugs in completion handline exposed only after the new Interpreter changes were merged. No other test failures in sight. I shall merge thee. |
Thanks everyone for the reviews. |
Revival of framed entity streaming.
Missing:
Resolves #18837 (honestly every HTTP demo I see wants this, also @RayRoestenburg's book 😉 )
Maybe this time it will get some review attention 😉