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.

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

[7.3] Restore discover histogram selection triggering fetch (#43097) #43287

Merged
merged 3 commits into from
Aug 16, 2019

Conversation

kertal
Copy link
Member

@kertal kertal commented Aug 14, 2019

Backports the following commits to 7.3:

* Listen to timefilter fetch event to start fetching

* Improve functional test to catch error
@elasticmachine
Copy link
Contributor

💔 Build Failed

@timroes
Copy link
Contributor

timroes commented Aug 15, 2019

Jenkins, test this

@elasticmachine
Copy link
Contributor

💔 Build Failed

}
expect(Math.round(newDurationHours)).to.be(3);
const rowData = await PageObjects.discover.getDocTableIndex(1);
expect(rowData).to.have.string('Sep 20, 2015 @ 02:51:53.327');

This comment was marked as outdated.

@elasticmachine
Copy link
Contributor

💔 Build Failed

});

it('should modify the time range when the histogram is brushed', async function () {
it.skip('should modify the time range when the histogram is brushed', async function () {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't get this working on 7.3 only, so skipping for now, since the fix itself seems to work without problems.

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@timroes timroes merged commit 0dc0f22 into elastic:7.3 Aug 16, 2019
@kertal
Copy link
Member Author

kertal commented Aug 16, 2019 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants