Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Comments

[O11y][Apache Spark] Add dimension mapping for application datastream#7991

Merged
harnish-crest-data merged 5 commits intoelastic:mainfrom
chavdaharnish:apache_spark-application-add-dimension
Oct 12, 2023
Merged

[O11y][Apache Spark] Add dimension mapping for application datastream#7991
harnish-crest-data merged 5 commits intoelastic:mainfrom
chavdaharnish:apache_spark-application-add-dimension

Conversation

@harnish-crest-data
Copy link
Contributor

  • Enhancement

What does this PR do?

This PR adds dimension fields for application data stream to support TSDB enablement.

Checklist

@elasticmachine
Copy link

elasticmachine commentedSep 27, 2023•
edited
Loading

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline ViewTest ViewChangesArtifactspreviewpreview

Expand to view the summary

Build stats

  • Start Time: 2023-10-12T10:13:56.149+0000

  • Duration: 20 min 5 sec

Test stats 🧪

TestResults
Failed0
Passed15
Skipped0
Total15

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@elasticmachine
Copy link

elasticmachine commentedSep 27, 2023•
edited
Loading

🌐 Coverage report

NameMetrics % (covered/total)Diff
Packages100.0% (0/0)💚
Files100.0% (0/0)💚
Classes100.0% (0/0)💚
Methods75.0% (12/16)❕
Lines100.0% (0/0)💚
Conditionals100.0% (0/0)💚

@harnish-crest-dataharnish-crest-data mentioned this pull requestSep 28, 2023
21 tasks
@harnish-crest-dataharnish-crest-data marked this pull request as ready for reviewSeptember 29, 2023 05:45
@harnish-crest-dataharnish-crest-data requested a review froma team as acode ownerSeptember 29, 2023 05:45
agithomas
agithomas previously approved these changesOct 5, 2023
Copy link
Contributor

@agithomasagithomas left a comment

Choose a reason for hiding this comment

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

LGTM!

@agithomas
Copy link
Contributor

@harnish-elastic , please complete the peer review additionally.

@agithomasagithomas dismissed theirstale reviewOctober 5, 2023 05:58

Errors identified

Copy link
Contributor

@agithomasagithomas left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Contributor

@milan-elasticmilan-elastic left a comment

Choose a reason for hiding this comment

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

LGTM!

…into apache_spark-application-add-dimensionConflicts:packages/apache_spark/changelog.ymlpackages/apache_spark/manifest.yml
@harnish-crest-dataharnish-crest-data merged commit92265bc intoelastic:mainOct 12, 2023
@elasticmachine
Copy link

Package apache_spark - 0.7.0 containing this change is available athttps://epr.elastic.co/search?package=apache_spark

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@agithomasagithomasagithomas approved these changes

+1 more reviewer

@milan-elasticmilan-elasticmilan-elastic approved these changes

Reviewers whose approvals may not affect merge requirements

Labels

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@harnish-crest-data@elasticmachine@agithomas@milan-elastic
[8]ページ先頭
©2009-2026 Movatter.jp