Skip to content

Conversation

@xrmx
Copy link
Owner

@xrmx xrmx commented Nov 25, 2024

Description

Fixes # (issue)

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

  • Test A

Does This PR Require a Contrib Repo Change?

  • Yes. - Link to PR:
  • No.

Checklist:

  • Followed the style guidelines of this project
  • Changelogs have been updated
  • Unit tests have been added
  • Documentation has been updated

@xrmx xrmx force-pushed the point-to-stable-branch-in-prerelease branch from 0d5d07e to ed3658a Compare November 26, 2024 10:23
@xrmx xrmx marked this pull request as ready for review November 26, 2024 10:23
@xrmx xrmx merged commit 5365a74 into main Nov 26, 2024
12 checks passed
xrmx pushed a commit that referenced this pull request May 15, 2025
…atchLogRecordProcessor can use (open-telemetry#4562)

* Refactor BatchLogRecordProcessor

* Respond to comments

* Fix lint

* Add delay for windows test.

* Fix fork test

* Initial Commit

* Another commit

* Fix lint / precommit

* Revert some old changes

* Fix lint issues

* Fix typo

* Fix lint and spellcheck

* Update test to use BatchLogRecordProcessor instead of BatchProcessor

* Add a sleep to see if it helps this test pass on pypy 3.8

* fix lint and precommit

* Add sleep to try to fix test..

* Fix flaky test attempt #2

* Fix test again..

* Fix test again

* Try again

* Fix again..

* Reintroduce weakref, I accidentlly undid that change in my last PR

* Add changelog

* Respond to comments on PR

* Make BatchProcessor a member of BLRP instead of having BLRP subclass it

* Run precommit

* Use a generic Protocol for the Exporter

* Minor change

* Fix bad changelog mearge

* Respond to comments..

---------

Co-authored-by: Leighton Chen <lechen@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants