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

to address #3918, reuse a container on applicationError #3941

Merged
merged 12 commits into from
Sep 5, 2018
Merged

to address #3918, reuse a container on applicationError #3941

merged 12 commits into from
Sep 5, 2018

Commits on Aug 24, 2018

  1. to address apache#3918, reuse a container when applicationError is re… 

    …turned during /run
    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    54bbbbf View commit details
    Browse the repository at this point in the history

  2. generate ApplicationResponse.containerError during failed init (inste… 

    …ad of ApplicationResponse.applicationError)
    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    4e690de View commit details
    Browse the repository at this point in the history

  3. update tests to check for ContainerError

    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    6961c99 View commit details
    Browse the repository at this point in the history

  4. update tests to check for ContainerError

    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    0309233 View commit details
    Browse the repository at this point in the history

  5. update tests to check for ContainerError

    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    99e39d2 View commit details
    Browse the repository at this point in the history

  6. assert destroyCount == 0 (for ApplicationError)

    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    06d1748 View commit details
    Browse the repository at this point in the history

  7. added comment for new test assertions

    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    3ede7cb View commit details
    Browse the repository at this point in the history

  8. timeout on run now produces ActivationResponse.containerError

    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    9b6e4b4 View commit details
    Browse the repository at this point in the history

  9. rename ActivationResponse.containerError -> `ActivationResponse.dev… 

    …eloperError`
    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    5cf9106 View commit details
    Browse the repository at this point in the history

  10. timeout on run now produces ActivationResponse.containerError (Acti… 

    …vationResponse.developerError)
    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    9407d47 View commit details
    Browse the repository at this point in the history

  11. timeout on run now produces ActivationResponse.containerError (Acti… 

    …vationResponse.developerError)
    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    94a6f48 View commit details
    Browse the repository at this point in the history

  12. timeout on run now produces ActivationResponse.containerError (Acti… 

    …vationResponse.developerError)
    @tysonnorris
    tysonnorris committed Aug 24, 2018
    Configuration menu
    Copy the full SHA
    6b9f24b View commit details
    Browse the repository at this point in the history