deepakpanda93 commented on issue #16705:
URL: https://github.com/apache/hudi/issues/16705#issuecomment-4892601929
This issue was reviewed as part of the JIRA-migrated backlog triage.
Findings: The current `TestSimpleSchemaConflictResolutionStrategy` no longer
extends `SimpleSchemaConflictResolutionStrategy`. On `master` (and in the 1.0.2
/ 1.1.0 release lines) it is a plain `public class
TestSimpleSchemaConflictResolutionStrategy { ... }` that uses Mockito `@Mock`
fields (e.g. mocked `config`, `viewManager`, `engineContext`) and holds the
strategy as a mocked collaborator rather than subclassing it — exactly the
improvement requested here. This mock-based form was introduced with PR #12781.
If you're still seeing the old subclass-based form on a recent version,
please reopen with details.
Closing as fixed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
