Skip to content

Update ConnectionFactoryProvider.java#2039

Open
christopherfreeman311-prog wants to merge 1 commit into
apache:mainfrom
christopherfreeman311-prog:patch-1
Open

Update ConnectionFactoryProvider.java#2039
christopherfreeman311-prog wants to merge 1 commit into
apache:mainfrom
christopherfreeman311-prog:patch-1

Conversation

@christopherfreeman311-prog
Copy link
Copy Markdown

No description provided.

@mattrpav mattrpav self-assigned this May 22, 2026
@mattrpav
Copy link
Copy Markdown
Contributor

What is the intent of this change?

@jbonofre
Copy link
Copy Markdown
Member

I'm a bit concerned by this PR:

  1. No description: as @mattrpav pointed out, we don't have a clear understanding of the purpose here
  2. The change itself doesn't make sense to me (just removing the connection factory in the provider)

Without a clear description of the intend, I think we should close this PR.

@jbonofre jbonofre self-requested a review May 22, 2026 03:38
@jbonofre jbonofre self-assigned this May 22, 2026
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.

3 participants