Skip to content

fix(connector): move plugin initialization into PluginLoader for lazy init#6932

Open
pingsutw wants to merge 1 commit intov2from
fix/connector-plugin-lazy-init
Open

fix(connector): move plugin initialization into PluginLoader for lazy init#6932
pingsutw wants to merge 1 commit intov2from
fix/connector-plugin-lazy-init

Conversation

@pingsutw
Copy link
Member

@pingsutw pingsutw commented Feb 23, 2026

Summary

  • Move clientSet, connectorRegistry, plugin struct creation, and watchConnectors call from newConnectorPlugin into the PluginLoader callback
  • This enables lazy initialization — connector clients are only created when the plugin is actually loaded, not at registration time
  • Use cfg.SupportedTaskTypes directly for the initial SupportedTaskTypes field instead of fetching from the registry at startup

Test plan

  • Existing unit tests pass for the connector plugin
  • Verify connector plugin loads correctly and registers supported task types at runtime

… init

Move clientSet, connectorRegistry, and plugin struct creation from
newConnectorPlugin into the PluginLoader callback to enable lazy
initialization. Also use cfg.SupportedTaskTypes directly instead of
fetching from the registry at startup time.

Signed-off-by: Kevin Su <pingsutw@apache.org>
@pingsutw pingsutw changed the base branch from master to v2 February 23, 2026 18:38
@github-actions github-actions bot mentioned this pull request Feb 13, 2026
3 tasks
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.

1 participant