apache / helix

Mirror of Apache Helix
Apache License 2.0
457 stars 218 forks source link

Fix incompatibility for customers which implement HelixAdmin and BaseDataAccessor #2809

Closed zpinto closed 1 month ago

zpinto commented 1 month ago

Issues

Description

After changing public interfaces, we did not add new default implementations for new methods. This is backwards incompatible. We are adding defaults in this PR that throw not implemented exception.

Tests

NA

Changes that Break Backward Compatibility (Optional)

(Consider including all behavior changes for public methods or API. Also include these changes in merge description so that other developers are aware of these changes. This allows them to make relevant code changes in feature branches accounting for the new method/API behavior.)

Documentation (Optional)

(Link the GitHub wiki you added)

Commits

Code Quality