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.

Sign up for GitHub

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

Jump to bottom

Jetty 9.4.x 4331 async close complete - take 2 #4398

Closed
wants to merge 18 commits into from
Closed

Jetty 9.4.x 4331 async close complete - take 2 #4398

wants to merge 18 commits into from

Commits on Dec 2, 2019

  1. Issue #4376 Async Content Complete 

    Added test harness to reproduce unready completing write.
Fixed test by not closing output prior to becoming READY

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 2, 2019
    Copy the full SHA
    ec969f4 View commit details
    Browse the repository at this point in the history

  2. Issue #4331 Async Write Complete 

    Test harness to reproduce unready when closing/completing.

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 2, 2019
    Copy the full SHA
    4592441 View commit details
    Browse the repository at this point in the history

  3. Issue #4331 Async Write Complete 

    test both PENDING and UNREADY

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 2, 2019
    Copy the full SHA
    be28e87 View commit details
    Browse the repository at this point in the history

Commits on Dec 3, 2019

  1. Issue #4331 Async Write Complete 

    test cleanups

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 3, 2019
    Copy the full SHA
    d686f58 View commit details
    Browse the repository at this point in the history

  2. Issue #4331 Async Close Complete 

    Cleanups of write

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 3, 2019
    Copy the full SHA
    41daa3f View commit details
    Browse the repository at this point in the history

  3. Merge remote-tracking branch 'origin/jetty-9.4.x' into jetty-9.4.x-43… 

    …31-asyncCloseComplete2
    @gregw
    gregw committed Dec 3, 2019
    Copy the full SHA
    49fd2ed View commit details
    Browse the repository at this point in the history

Commits on Dec 4, 2019

  1. WIP 

    Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 4, 2019
    Copy the full SHA
    85eaa1f View commit details
    Browse the repository at this point in the history

  2. Issue #4331 Close Complete 

    Work in progress

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 4, 2019
    Copy the full SHA
    a699aa0 View commit details
    Browse the repository at this point in the history

Commits on Dec 5, 2019

  1. Issue #4331 Close Complete 

    Added async close to HttpWriter and ResponseWriter
Always use async close, with blocker if necessary.

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 5, 2019
    Copy the full SHA
    186164c View commit details
    Browse the repository at this point in the history

  2. Issue #4331 Close Complete 

    Working async close complete!

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 5, 2019
    Copy the full SHA
    88398c8 View commit details
    Browse the repository at this point in the history

  3. Issue #4331 Close Complete 

    invert test as we can now call complete when not ready!

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 5, 2019
    Copy the full SHA
    cfc63a9 View commit details
    Browse the repository at this point in the history

  4. Issue #4331 Close Complete 

    fixed transition to ERROR state

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 5, 2019
    Copy the full SHA
    40ebd16 View commit details
    Browse the repository at this point in the history

  5. Issue #4331 Close Complete 

    async close after onError

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 5, 2019
    Copy the full SHA
    b5971a6 View commit details
    Browse the repository at this point in the history

Commits on Dec 6, 2019

  1. Issue #4331 Close Complete 

    minor cleanups

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 6, 2019
    Copy the full SHA
    d53194b View commit details
    Browse the repository at this point in the history

  2. Issue #4331 Close Complete 

    Fix for proxy tests

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 6, 2019
    Copy the full SHA
    8340a20 View commit details
    Browse the repository at this point in the history

Commits on Dec 8, 2019

  1. Issue #4331 Close Complete 

    Fix write loop to handle clear of p=0,l=0 rather than p=l

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 8, 2019
    Copy the full SHA
    ca0e0c4 View commit details
    Browse the repository at this point in the history

Commits on Dec 9, 2019

  1. Issue #4331 Close Complete 

    Removed old close on all content mechanism
Cleanups and some more TODOs

Signed-off-by: Greg Wilkins <gregw@webtide.com>
    @gregw
    gregw committed Dec 9, 2019
    Copy the full SHA
    1da0062 View commit details
    Browse the repository at this point in the history

  2. Merged branch 'jetty-9.4.x' into 'jetty-9.4.x-4331-asyncCloseComplete2'.

    @sbordet
    sbordet committed Dec 9, 2019
    Copy the full SHA
    03e4a7b View commit details
    Browse the repository at this point in the history