Skip to content

[refactor][fn] Use Map instead of TreeMap for connector/function API types - #25790

Merged
lhotari merged 1 commit into
apache:masterfrom
Dream95:refactor_fn_treemap
May 16, 2026
Merged

lhotari merged 1 commit into
apache:masterfrom
Dream95:refactor_fn_treemap

Conversation

@Dream95

@Dream95 Dream95 commented May 15, 2026 •

Copy link
Copy Markdown
Contributor

Closes #25782

Motivation

Connector- and function-related code unnecessarily used TreeMap as the declared type for fields, parameters, return values, and local variables.
This refactor follows up on #25773 (incremental built-in connector reload), as tracked in issue #25782.

Modifications

ConnectorsManager / FunctionsManager
ReloadConnectorsResult
ConnectorUtils/FunctionUtils
LocalRunner
ConnectorUtilsReloadTest

Verifying this change

  • Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

@lhotari lhotari left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@lhotari lhotari changed the title [refactor][fn] Use Map instead of TreeMap for connector/function API types (#25782) [refactor][fn] Use Map instead of TreeMap for connector/function API types May 15, 2026
@lhotari lhotari added this to the 5.0.0-M1 milestone May 15, 2026
@lhotari
lhotari merged commit eba873d into apache:master May 16, 2026
46 of 48 checks passed
lhotari pushed a commit that referenced this pull request May 18, 2026
lhotari pushed a commit that referenced this pull request May 18, 2026
@Dream95
Dream95 deleted the refactor_fn_treemap branch May 25, 2026 03:39
nodece pushed a commit to ascentstream/pulsar that referenced this pull request May 27, 2026
priyanshu-ctds pushed a commit to datastax/pulsar that referenced this pull request Jun 9, 2026
…types (apache#25790)

Signed-off-by: Dream95 <[email protected]>
(cherry picked from commit eba873d)
(cherry picked from commit 95c8608)
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.

[refactor] Connection code unnecessary uses TreeMap as parameter, field, or variable type

3 participants