-
Notifications
You must be signed in to change notification settings - Fork 8k
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
Support for date_nanos type timestamps in context view #38023
Conversation
d90fc25
to
ffb3345
Compare
- too much shared code
- merge fetchPredecessorRows & fetchSuccessorRows - new function fetchSurroundingRows - less code duplication
d7cd763
to
a1e5d10
Compare
💚 Build Succeeded |
Pinging @elastic/kibana-app |
I'm currently improving code, moving to typescript, but I think it's wiser to review that in a different PR. So this PR is focused on fixing the rounding issue of |
src/legacy/core_plugins/kibana/public/context/api/__tests__/successors.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code LGTM, added two questions but nothing important
thank you so much for reviewing, I'll continue then finishing the typescript version, which will be a separate PR |
Now using the the right iso timestamp in every test case
Now using the right ISO timestamp in every test case
💔 Build Failed |
💚 Build Succeeded |
💚 Build Succeeded |
Note: this wasn't backported, what was backported was #38735, the typescript converted version. |
Summary
Provide ES
date_nanos
support in context view. This is part of #31424.In #36111 date_nanos support was implemented, but the context view was disabled, due to issues with rounded sort values.
Testing
You can provision date_nanos data by:
Or you can convert demo test data to date nanos with the following instruction:
#36111 (comment)
Checklist
Use
strikethroughsto remove checklist items you don't feel are applicable to this PR.- [ ] Any text added follows EUI's writing guidelines, uses sentence case text and includes [i18n support](https://github.com/elastic/kibana/blob/master/packages/kbn-i18n/README.md)- [ ] Documentation was added for features that require explanation or tutorials- [ ] This was checked for keyboard-only and screenreader accessibilityFor maintainers
- [ ] This was checked for breaking API changes and was labeled appropriately