Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Improve][Document] Update the guide and add example #7388

Merged
merged 33 commits into from
Aug 22, 2024
Merged

Conversation

yujian225
Copy link
Contributor

Update [deployment.md]
Update [quick-start-seatunnel-engine.md] Added an extended example (mysql to doris)

@yujian225 yujian225 changed the title Modify the document [start-v2/locally] [start-v2/locally]Modify the document Aug 14, 2024
@Hisoka-X
Copy link
Member

Please update english doc too. Thanks.

@Hisoka-X Hisoka-X changed the title [start-v2/locally]Modify the document [Improve][Document] Update the guide and add example Aug 15, 2024
@yujian225
Copy link
Contributor Author

请也更新英文文档。谢谢。

已更新

```

If you want the example application to work properly, you need to add the following plugins.
Typically, you do not need all the connector plugins. You can specify the required plugins by configuring `config/plugin_config`. For example, if you want the sample application to work properly, you will need the `connector-console` and `connector-fake` plugins. You can modify the `plugin.properties` configuration file as follows:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
Typically, you do not need all the connector plugins. You can specify the required plugins by configuring `config/plugin_config`. For example, if you want the sample application to work properly, you will need the `connector-console` and `connector-fake` plugins. You can modify the `plugin.properties` configuration file as follows:
Typically, you do not need all the connector plugins. You can specify the required plugins by configuring `config/plugin_config`. For example, if you want the sample application to work properly, you will need the `connector-console` and `connector-fake` plugins. You can modify the `plugin_config` configuration file as follows:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The issue mentioned above has been resolved.

Comment on lines 37 to 40
If you need a specific connector version, taking 2.3.7 as an example, you need to execute the following command:

```bash
sh bin/install-plugin.sh 2.3.7
```
Copy link
Member

Choose a reason for hiding this comment

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

why delete this part?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The issue mentioned above has been resolved.

@Hisoka-X
Copy link
Member

Please follow the guide to open github action on your fork repository. Thanks
image

Comment on lines 158 to 162
:::tip

Starting from version 2.3.1, the `-e` parameter in `seatunnel.sh` has been deprecated. Please use the `-m` parameter instead.

:::
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
:::tip
Starting from version 2.3.1, the `-e` parameter in `seatunnel.sh` has been deprecated. Please use the `-m` parameter instead.
:::

Comment on lines 156 to 160
:::tip

从2.3.1版本开始,seatunnel.sh中的-e参数被废弃,请改用-m参数。

:::
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
:::tip
从2.3.1版本开始,seatunnel.sh中的-e参数被废弃,请改用-m参数。
:::

Copy link
Member

@Hisoka-X Hisoka-X left a comment

Choose a reason for hiding this comment

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

Thanks @yujian225

@hailin0 hailin0 merged commit 708a72c into apache:dev Aug 22, 2024
7 checks passed
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.

3 participants