Skip to content

[5.x] Tagcloud fixes#9246

Merged
epixa merged 1 commit into5.xfrom
jasper/backport/9194/5.x
Nov 29, 2016
Merged

[5.x] Tagcloud fixes#9246
epixa merged 1 commit into5.xfrom
jasper/backport/9194/5.x

Conversation

@elastic-jasper
Copy link
Copy Markdown
Contributor

Tagcloud fixes

Backports PR #9194

Commit 1:
Improve robustness

Tagcloud was subsceptible to race conditions, potentially yielding stale clouds.

  • Introduced queue to explicitly keep track of outstanding jobs and avoid clobbering of the state.
  • Expanded unit tests

Commit 2:
ensure tagcloud expands to full height in reporting

Commit 3:
remove magic numbers

Backports PR #9194

**Commit 1:**
Improve robustness

Tagcloud was subsceptible to race conditions, potentially yielding stale clouds.
- Introduced queue to explicitly keep track of outstanding jobs and avoid clobbering of the state.
- Expanded unit tests

* Original sha: af5d5c6
* Authored by Thomas Neirynck <thomas@elastic.co> on 2016-11-23T00:20:04Z

**Commit 2:**
ensure tagcloud expands to full height in reporting

* Original sha: de95bf1
* Authored by Thomas Neirynck <thomas@elastic.co> on 2016-11-28T23:13:18Z

**Commit 3:**
remove magic numbers

* Original sha: 011474c
* Authored by Thomas Neirynck <thomas@elastic.co> on 2016-11-28T23:33:48Z
@elastic-jasper elastic-jasper added the backport This PR is a backport of another PR label Nov 29, 2016
@epixa epixa merged commit ece9e03 into 5.x Nov 29, 2016
@epixa epixa deleted the jasper/backport/9194/5.x branch November 29, 2016 16:49
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.

2 participants