Skip to content

Conversation

@hanicz
Copy link
Contributor

@hanicz hanicz commented Jan 7, 2026

…case

KNOX-3235 - Rename DBDataSource classes to closely follow Factory pattern

What changes were proposed in this pull request?

  • Renamed the DataSource classed to Factory so they correctly represent their reponsibility. The previous Factory class is renamed to Provider.
  • Fixed PMD suppress on the switch-case.

How was this patch tested?

Build (clean verify), unit tests,

Copy link
Contributor

@smolnar82 smolnar82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link

github-actions bot commented Jan 7, 2026

Test Results

7 tests   7 ✅  1s ⏱️
1 suites  0 💤
1 files    0 ❌

Results for commit 2148a84.

@hanicz hanicz merged commit 8d5cbdd into apache:master Jan 7, 2026
3 checks passed
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