Skip to content

Improve MetricsReporter loading with class loader fallback#1

Closed
bk-mz wants to merge 19 commits intomainfrom
fixing_metrics_reporter_classloader
Closed

Improve MetricsReporter loading with class loader fallback#1
bk-mz wants to merge 19 commits intomainfrom
fixing_metrics_reporter_classloader

Conversation

@bk-mz
Copy link
Owner

@bk-mz bk-mz commented Jun 7, 2024

manuzhang and others added 19 commits March 11, 2024 11:32
1. Fix internal links
2. Remove period in title
3. Fix numbered list with code blocks
4. Add identifier fields as an approach for upsert mode
…he#9911)

Bumps [org.roaringbitmap:RoaringBitmap](https://github.com/RoaringBitmap/RoaringBitmap) from 1.0.1 to 1.0.5.
- [Release notes](https://github.com/RoaringBitmap/RoaringBitmap/releases)
- [Commits](RoaringBitmap/RoaringBitmap@1.0.1...1.0.5)

---
updated-dependencies:
- dependency-name: org.roaringbitmap:RoaringBitmap
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Site: Update release notes for 1.5.0

* Address comments

* add view PRs
1. Fix internal links
2. Move `Type Compatibility` section from spark-writes.md to spark-getting-started.md
- Resolve Nessie version for site docs
- Add release notes link to template
- Add fallback to thread's context class loader when loading MetricsReporter in CatalogUtil.
- Introduce tryLoadCtor method to handle constructor loading.
- Add a test case to validate fallback mechanism using a custom MetricsReporter.

Enhances flexibility in loading custom MetricsReporter implementations.
@bk-mz bk-mz marked this pull request as draft June 7, 2024 06:27
@bk-mz bk-mz closed this Jun 7, 2024
@bk-mz bk-mz deleted the fixing_metrics_reporter_classloader branch June 7, 2024 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants