Skip to content

[Doc][Contribution] The development-environment-setup documentation is incorrect about how to configure MySQL as the default database#14288

Closed
QuakeWang wants to merge 1 commit intoapache:devfrom
QuakeWang:issue-14275
Closed

[Doc][Contribution] The development-environment-setup documentation is incorrect about how to configure MySQL as the default database#14288
QuakeWang wants to merge 1 commit intoapache:devfrom
QuakeWang:issue-14275

Conversation

@QuakeWang
Copy link
Contributor

Purpose of the pull request

close: #14275

Modify the current documentation regarding mysql-connector-java configuration errors.

Comment on lines +184 to +191
- If you use MySQL as meta-database, you need to add `mysql-connector-java` dependency in `pom.xml` file which is under `dolphinscheduler-api` and `dolphinscheduler-master` modules, it is not needed if you use PostgreSQL, the detailed configuration is as follows:
```xml
<dependency>
<groupId>mysql</groupId>
<artifactId>mysql-connector-java</artifactId>
<scope>compile</scope>
</dependency>
```
Copy link
Member

Choose a reason for hiding this comment

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

Hi @QuakeWang currently we can simple comment the mysql dependence scope in pom.xml in module dolphinscheduler-bom

<dependency>
    <groupId>mysql</groupId>
    <artifactId>mysql-connector-java</artifactId>
    <version>${mysql-connector.version}</version>
    <scope>test</scope>
</dependency>

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @QuakeWang currently we can simple comment the mysql dependence scope in pom.xml in module dolphinscheduler-bom

<dependency>
    <groupId>mysql</groupId>
    <artifactId>mysql-connector-java</artifactId>
    <version>${mysql-connector.version}</version>
    <scope>test</scope>
</dependency>

Does this mean that mysql-connector will work just by changing the pom file which is under dolphinscheduler-bom?

@zhongjiajie zhongjiajie added this to the 3.1.8 milestone Jun 13, 2023
@zhuangchong zhuangchong modified the milestones: 3.1.8, 3.1.9 Aug 15, 2023
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 7 days if no further activity occurs.

@github-actions github-actions bot added the Stale label Dec 14, 2023
@zhongjiajie zhongjiajie removed the Stale label Dec 20, 2023
@zhuangchong zhuangchong modified the milestones: 3.1.9, 3.1.10 Dec 26, 2023
@QuakeWang QuakeWang closed this Feb 20, 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.

[Doc][Contribution] The development-environment-setup documentation is incorrect about how to configure MySQL as the default database.

3 participants