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

[heatmap] Fix x time scale with calendar intervals #117619

Merged
merged 5 commits into from
Nov 9, 2021

Conversation

markov00
Copy link
Member

@markov00 markov00 commented Nov 5, 2021

Summary

The PR will introduce a fix from elastic-charts add the ability to use both calendars and fixed intervals as described in Elasticserach date_histogram aggregation

This fix was required to cover the Lens usage when calendar intervals are used in conjunction with the date_histogram aggs.

In ML Swimlane, the fix only covers the API changes in elastic-charts but doesn't cause any differences in the chart behaviour.

To do:

@markov00 markov00 force-pushed the 2021_11_03-fix_heatmap branch from 3ead173 to 12cf9ff Compare November 9, 2021 09:47
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
apm 2.7MB 2.7MB +1.0B
lens 953.8KB 954.3KB +520.0B
ml 3.5MB 3.5MB +106.0B
total +627.0B

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
core 267.2KB 267.6KB +444.0B
kbnUiSharedDeps-npmDll 5.0MB 5.0MB +5.5KB
total +5.9KB

History

  • 💔 Build #4913 failed 3ead173d387f79f46a2f7287a3913fdac50d1bda
  • 💔 Build #4562 failed 4e3487bc9689d43b8ce0d0659445c799f9ca3908

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@markov00 markov00 added auto-backport Deprecated - use backport:version if exact versions are needed bug Fixes for quality problems that affect the customer experience Feature:ElasticCharts Issues related to the elastic-charts library release_note:skip Skip the PR/issue when compiling release notes Team:DataVis Team label for DataVis Team v8.0.0 v8.1.0 labels Nov 9, 2021
@markov00 markov00 marked this pull request as ready for review November 9, 2021 12:34
@markov00 markov00 requested review from a team as code owners November 9, 2021 12:34
@elasticmachine
Copy link
Contributor

Pinging @elastic/datavis (Feature:ElasticCharts)

Copy link
Contributor

@dej611 dej611 left a comment

Choose a reason for hiding this comment

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

Code review 👍
Tested locally on Safari ✅

Copy link
Contributor

@darnautov darnautov left a comment

Choose a reason for hiding this comment

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

Swim lane changes LGTM

@markov00 markov00 merged commit 687b97b into elastic:main Nov 9, 2021
@kibanamachine
Copy link
Contributor

💔 Backport failed

Status Branch Result
8.0 Commit could not be cherrypicked due to conflicts

To backport manually run:
node scripts/backport --pr 117619

markov00 added a commit to markov00/kibana that referenced this pull request Nov 9, 2021
The commit fixes the Lens heatmap time axis when used with calendars time intervals in date_histogram aggs
# Conflicts:
#	package.json
#	yarn.lock
markov00 added a commit that referenced this pull request Nov 9, 2021
…118065)

* [heatmap] Fix x time scale with calendar intervals (#117619)

The commit fixes the Lens heatmap time axis when used with calendars time intervals in date_histogram aggs
# Conflicts:
#	package.json
#	yarn.lock

* commit using @elastic.co
markov00 added a commit to nickofthyme/kibana that referenced this pull request Dec 20, 2021
The commit fixes the Lens heatmap time axis when used with calendars time intervals in date_histogram aggs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-backport Deprecated - use backport:version if exact versions are needed bug Fixes for quality problems that affect the customer experience Feature:ElasticCharts Issues related to the elastic-charts library release_note:skip Skip the PR/issue when compiling release notes Team:DataVis Team label for DataVis Team v8.0.0 v8.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants