Skip to content

[backport] PR #6651 to 4.x#6673

Merged
spalger merged 1 commit into4.xfrom
jasper/backport/6651/4.x
Mar 28, 2016
Merged

[backport] PR #6651 to 4.x#6673
spalger merged 1 commit into4.xfrom
jasper/backport/6651/4.x

Conversation

@elastic-jasper
Copy link
Copy Markdown
Contributor

Backport PR #6651

Commit 1:
[courier/segmentedRequest] mark the request "started" synchronously

Commit 2:
[courier/fetch] detect aborted requests more agressively

Commit 3:
[courier/callClient] if there are no executable requests, do nothing

Commit 4:
[courier/tests] updated test

@elastic-jasper elastic-jasper added backport This PR is a backport of another PR has conflicts labels Mar 28, 2016
---------

**Commit 1:**
[courier/segmentedRequest] mark the request "started" synchronously

* Original sha: 8a49926
* Authored by spalger <spalger@users.noreply.github.com> on 2016-03-24T21:35:00Z

**Commit 2:**
[courier/fetch] detect aborted requests more agressively

* Original sha: c12f18d
* Authored by spalger <spalger@users.noreply.github.com> on 2016-03-24T21:35:46Z

**Commit 3:**
[courier/callClient] if there are no executable requests, do nothing

* Original sha: 8fef03b
* Authored by spalger <spalger@users.noreply.github.com> on 2016-03-24T21:36:18Z

**Commit 4:**
[courier/tests] updated test

* Original sha: 985f997
* Authored by spalger <spalger@users.noreply.github.com> on 2016-03-24T22:25:31Z
@epixa epixa force-pushed the jasper/backport/6651/4.x branch from b4a31b9 to 6ac2c36 Compare March 28, 2016 15:20
@epixa
Copy link
Copy Markdown
Contributor

epixa commented Mar 28, 2016

@spalger Can you review this? I basically had to apply the backport by hand since courier differs so much between master and 4.x.

@epixa epixa assigned spalger and unassigned epixa Mar 28, 2016
@spalger
Copy link
Copy Markdown
Contributor

spalger commented Mar 28, 2016

LGTM

@spalger spalger merged commit baa1d8a into 4.x Mar 28, 2016
@epixa epixa deleted the jasper/backport/6651/4.x branch March 28, 2016 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport This PR is a backport of another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants