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

Documentation for MySQL and PostgreSQL in MP #9794

Open
hyder opened this issue Feb 13, 2025 · 0 comments
Open

Documentation for MySQL and PostgreSQL in MP #9794

hyder opened this issue Feb 13, 2025 · 0 comments
Labels
4.x Version 4.x docs enhancement New feature or request jpa/jta P3

Comments

@hyder
Copy link

hyder commented Feb 13, 2025

Environment Details

  • Helidon Version: 4.1.6
  • Helidon MP

Enhancement Description

Helidon MP persistence documentation lists only H2 and Oracle database drivers.

<dependency>
    <groupId>io.helidon.integrations.db</groupId>
    <artifactId>h2</artifactId>
    <scope>runtime</scope> 
</dependency>

<dependency>
    <groupId>io.helidon.integrations.db</groupId>
    <artifactId>ojdbc</artifactId>
    <scope>runtime</scope> 
</dependency>

This might give the wrong impression that only the above 2 databases are supported whereas the following drivers are also available:

        <dependency>
            <groupId>io.helidon.integrations.db</groupId>
            <artifactId>helidon-integrations-db-pgsql</artifactId>
        </dependency>
        <dependency>
            <groupId>io.helidon.integrations.db</groupId>
            <artifactId>helidon-integrations-db-mysql</artifactId>
        </dependency>

Please add the above two to the documentation for MP.

It's also unclear if the above works for MP only e.g. in the SE documentation, instead of using

<dependency>
    <groupId>io.helidon.integrations.db</groupId>
    <artifactId>h2</artifactId>
    <scope>runtime</scope> 
</dependency>

for H2 database, the documentation suggests the followingg:

     <dependency>
         <groupId>com.h2database</groupId> 
         <artifactId>h2</artifactId>
    </dependency>

If the jdbc drivers are already in io.helidon.ingrations.db, why would we need to add com.h2database? Please clarify

@hyder hyder added the enhancement New feature or request label Feb 13, 2025
@github-project-automation github-project-automation bot moved this to Triage in Backlog Feb 13, 2025
@m0mus m0mus moved this from Triage to Normal priority in Backlog Feb 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
4.x Version 4.x docs enhancement New feature or request jpa/jta P3
Projects
Status: Normal priority
Development

No branches or pull requests

2 participants