Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

This page is meant as a template for writing a KIP. To create a KIP choose Tools->Copy on this page and modify with your content and replace the heading with the next KIP number and a description of your issue. Replace anything in italics with your own description.

Status

Current state: Under Discussion

Discussion thread: here TODO

JIRA: hereTODO

Please keep the discussion on the mailing list rather than commenting on the wiki (wiki discussions get unwieldy fast).

Motivation

Connect has a REST endpoint for Workers, which today have resources for managing "plugins" (class `ConnectorPluginsResource`) and "connectors" (class `ConnectorsResource`). The `plugins` resource is particularly useful to query the Connect cluster which connectors it has loaded to its plugin path ("GET /connector-pluings" endpoint). In reality, Connectors are just one type of plugin: Other types include Simple Message Transformations (SMTs), Converters, HeaderConverters, ConfigProviders, and RestExtensions (loaded by the "DelegatingClassLoader").

Since SMTs are a popular feature, and the current way to know which are loaded is to read the logs. In many cases, logs may be cleaned and a long-running server will have no way to conveniently reporting loaded SMTs. Only GET requests will be in scope.

This KIP will add HTTP endpoints for Transformations, Converters, HeaderConverters, ConfigProviders, and ConnectRestExtensions to Connect RestServers.

Public Interfaces

The new endpoints are in bold below:

/connector-plugins

...

get:

...

summary:

...

Lists

...

available

...

Connector plugins
...
...
/connector-plugins/{connectorType}/config/validate
put:
summary: Validate the supplied connector config
...
...

/plugins

  get:

    summary: Returns plugin descriptions of all types

....

/plugins/{pluginType}:

  get:

    summary: Returns plugin descriptions of the given type

    parameters:

      -in: path

        name: pluginType

        schema:

          type: string

        required: true

        description: pluginType can be one of Connector, Converter, HeaderConverter, Transformation, ConfigProvider, ConnectRestExtension

....

Return types for 200OK responses will be same schemas as existing return types for existing endpoint "GET /connector-plugins".

Proposed Changes

The new endpoints are in bold below:described in the API Spec above.

The proposal is to implement them on a new "PluginsResource" analogous to the existing "ConnectorPluginsResource", and using the ConnectorPluginsResource's access to a Herder instance, by using Herder.plugins(), analogous to existing ConnectorPluginsResource.getConnectorPlugins(). 

Compatibility, Deprecation, and Migration Plan

This is a simple addition, so no compatibility or migration issues. Existing APIs are intentionally left unaltered for simplicity.

We will need to update the docs.

Rejected Alternatives

Limiting scope of KIP to just SMTs

The scope could be limited to just adding HTTP Endpoints for just SMTs, which have the most pressing need. However, any design extending the current API to include SMTs would be designed to be elegantly extensible, and then most of the work to just include the other Plugin types is already done.

Updating existing endpoint for "GET /connector-plugins/" to include all plugins, as its name suggests

This strategy would break compatibility, and there are suitable alternatives without breaking changes.

Add implementation on existing ConnectorsPluginsResource

This idea is good, too, but leaned towards a new class to keep the naming very precise.