You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "rmoff (via GitHub)" <gi...@apache.org> on 2024/12/02 10:05:52 UTC
[PR] [FLINK-36827] CDC quickstart tidyup [flink-cdc]
rmoff opened a new pull request, #3771:
URL: https://github.com/apache/flink-cdc/pull/3771
This makes some small improvements to the structure of the quickstart to make it easier to follow, and fixes a few nits (e.g. capitalization). It also adds the use of `privileged: true` flag in the Docker Compose for Mac.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [FLINK-36827] CDC quickstart tidyup [flink-cdc]
Posted by "Thorne-coder (via GitHub)" <gi...@apache.org>.
Thorne-coder commented on code in PR #3771:
URL: https://github.com/apache/flink-cdc/pull/3771#discussion_r1867260586
##########
docs/content/docs/get-started/quickstart/mysql-to-doris.md:
##########
@@ -104,25 +98,26 @@ Then `exit` exits and creates the Doris Docker cluster.
- MYSQL_PASSWORD=mysqlpw
```
-The Docker Compose should include the following services (containers):
-- MySQL: include a database named `app_db`
-- Doris: to store tables from MySQL
+ The Docker Compose should include the following services (containers):
+ - MySQL: include a database named `app_db`
+ - Doris: to store tables from MySQL
-To start all containers, run the following command in the directory that contains the `docker-compose.yml` file.
+3. Start all containers by running the following command in the directory that contains the `docker-compose.yml` file:
```shell
docker-compose up -d
```
-This command automatically starts all the containers defined in the Docker Compose configuration in a detached mode. Run docker ps to check whether these containers are running properly. You can also visit [http://localhost:8030/](http://localhost:8030/) to check whether Doris is running.
-#### Prepare records for MySQL
+ This command automatically starts all the containers defined in the Docker Compose configuration in a detached mode. Run `docker compose ps` to check whether these containers are running properly. You can also visit [http://localhost:8030/](http://localhost:8030/) to check whether Doris is running.
+
+### Prepare records for MySQL
1. Enter MySQL container
```shell
docker-compose exec mysql mysql -uroot -p123456
```
-2. create `app_db` database and `orders`,`products`,`shipments` tables, then insert records
+2. Create `app_db` database and `orders`,`products`,`shipments` tables
Review Comment:
Should we put all the tables create table sql statements here?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [FLINK-36827] CDC quickstart tidyup [flink-cdc]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #3771:
URL: https://github.com/apache/flink-cdc/pull/3771#issuecomment-3025885289
This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 60 days if no further activity occurs.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [FLINK-36827] CDC quickstart tidyup [flink-cdc]
Posted by "rmoff (via GitHub)" <gi...@apache.org>.
rmoff closed pull request #3771: [FLINK-36827] CDC quickstart tidyup
URL: https://github.com/apache/flink-cdc/pull/3771
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [FLINK-36827] CDC quickstart tidyup [flink-cdc]
Posted by "Sukang1002 (via GitHub)" <gi...@apache.org>.
Sukang1002 commented on PR #3771:
URL: https://github.com/apache/flink-cdc/pull/3771#issuecomment-2693333417
@rmoff Hello.Would you like to know if you are working on the Chinese version of this article? If not, can I update this Chinese version to the latest version?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [FLINK-36827] CDC quickstart tidyup [flink-cdc]
Posted by "Thorne-coder (via GitHub)" <gi...@apache.org>.
Thorne-coder commented on PR #3771:
URL: https://github.com/apache/flink-cdc/pull/3771#issuecomment-2513870825
I recommend u use new branch to fix without master branch
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [FLINK-36827] CDC quickstart tidyup [flink-cdc]
Posted by "Sukang1002 (via GitHub)" <gi...@apache.org>.
Sukang1002 commented on PR #3771:
URL: https://github.com/apache/flink-cdc/pull/3771#issuecomment-2693790611
Ok, thank you for your reply, then I will further improve this document.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [FLINK-36827] CDC quickstart tidyup [flink-cdc]
Posted by "rmoff (via GitHub)" <gi...@apache.org>.
rmoff commented on PR #3771:
URL: https://github.com/apache/flink-cdc/pull/3771#issuecomment-2693777787
@Sukang1002 I'm not planning to progress this PR further.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [FLINK-36827] CDC quickstart tidyup [flink-cdc]
Posted by "Thorne-coder (via GitHub)" <gi...@apache.org>.
Thorne-coder commented on PR #3771:
URL: https://github.com/apache/flink-cdc/pull/3771#issuecomment-2513856685
Thanks for your contribution!
The latest branch is currently version 3.2. x. Should we update the connector version of the document, as we no longer need to manually create a database in the Apache Doris database.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org