close
Skip to content

[Improve][Doc] Improve the connector-v2 develop doc#8190

Merged
dailai merged 2 commits into
apache:devfrom
YOMO-Lee:develop-doc-7959
Dec 3, 2024
Merged

[Improve][Doc] Improve the connector-v2 develop doc#8190
dailai merged 2 commits into
apache:devfrom
YOMO-Lee:develop-doc-7959

Conversation

@YOMO-Lee

@YOMO-Lee YOMO-Lee commented Dec 2, 2024

Copy link
Copy Markdown
Contributor

Supplementary explanations have been provided for the methods and classes that must be implemented during the source development process

…che#7959

Supplementary explanations have been provided for the methods and classes that must be implemented during the source development process

@Hisoka-X Hisoka-X 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.

Thanks @YOMO-Lee . This PR looks great.

Comment thread seatunnel-connectors-v2/README.md Outdated
Comment on lines +59 to +63
6.There are several classes that must be implemented on the source side, namely {connector name} Source, {connector name} SourceFactory, {connector name} SourceReader. Please refer to other connectors for details

7.{Connector Name} SourceFactory needs to be annotated with the **@AutoService (Factory.class)** annotation on the class name, and in addition to the required methods, an additional 'creatSource' method needs to be rewritten

8.{Connector Name} Source needs to override the getProducedCatalogTables method

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.

Please add sink too

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

But I have only done source development and not sink development, so I am not sure if what I am writing is correct

liunaijie
liunaijie previously approved these changes Dec 3, 2024
hailin0
hailin0 previously approved these changes Dec 3, 2024
@YOMO-Lee YOMO-Lee dismissed stale reviews from hailin0 and liunaijie via 5cffa18 December 3, 2024 02:00
@Hisoka-X Hisoka-X changed the title [hotfix] [DOC] Fix missing areas in the development documentation #7959 [Improve][Doc] Improve the connector-v2 develop doc Dec 3, 2024
@YOMO-Lee

YOMO-Lee commented Dec 3, 2024

Copy link
Copy Markdown
Contributor Author

@hailin0 @liunaijie please review again ,thx

@dailai dailai merged commit 8e8c534 into apache:dev Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants