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

[improve][ci] Upload native crash dump files hs_err_*.log & core.* in CI unit tests #19251

Merged
merged 1 commit into from
Jan 17, 2023

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Jan 17, 2023

Motivation

Modifications

  • upload native crash dump files hs_err_.log & core. in CI unit tests

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

- helps investigate issues when there's a JVM crash such as apache#19250
@lhotari lhotari added this to the 2.12.0 milestone Jan 17, 2023
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Jan 17, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #19251 (9c09d6b) into master (334c3a5) will increase coverage by 0.32%.
The diff coverage is 8.28%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master   #19251      +/-   ##
============================================
+ Coverage     47.04%   47.36%   +0.32%     
- Complexity     9190    18904    +9714     
============================================
  Files           607     1625    +1018     
  Lines         57677   132275   +74598     
  Branches       6007    14562    +8555     
============================================
+ Hits          27132    62650   +35518     
- Misses        27598    63312   +35714     
- Partials       2947     6313    +3366     
Flag Coverage Δ
unittests 47.36% <8.28%> (+0.32%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...rg/apache/pulsar/broker/delayed/bucket/Bucket.java 0.00% <0.00%> (ø)
...r/delayed/bucket/BucketDelayedDeliveryTracker.java 0.00% <0.00%> (ø)
...layed/bucket/CombinedSegmentDelayedIndexQueue.java 0.00% <0.00%> (ø)
...ulsar/broker/delayed/bucket/DelayedIndexQueue.java 0.00% <0.00%> (ø)
.../pulsar/broker/delayed/bucket/ImmutableBucket.java 0.00% <0.00%> (ø)
...he/pulsar/broker/delayed/bucket/MutableBucket.java 0.00% <0.00%> (ø)
...ed/bucket/TripleLongPriorityDelayedIndexQueue.java 0.00% <0.00%> (ø)
...rg/apache/pulsar/broker/service/BrokerService.java 57.44% <42.85%> (-0.43%) ⬇️
...che/pulsar/websocket/AbstractWebSocketHandler.java 39.28% <50.00%> (ø)
.../org/apache/pulsar/websocket/WebSocketService.java 48.96% <100.00%> (ø)
... and 1064 more

@lhotari lhotari merged commit 1bb7b31 into apache:master Jan 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ci doc-not-needed Your PR changes do not impact docs ready-to-test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants