DTStack/chunjun

[Bug] [SqlserverDynamicTableFactory.java ] there is a bug in getDialect function, this function will init driver with Microsoft JDBC driver.

Open

#1.087 geöffnet am 24. Juli 2022

Auf GitHub ansehen
 (2 Kommentare) (0 Reaktionen) (1 zugewiesene Person)Java (1.668 Forks)batch import
buggood first issue

Repository-Metriken

Stars
 (3.752 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

Search before asking

  • I had searched in the issues and found no similar issues.

What happened

I am using sql job to get data from sqlserver. After config the job, I got error "No suitable driver found for jdbc:jtds:sqlserve".

After checking the init way of sqlserver-x, I found the getDialect function with some bugs.

    @Override
    protected JdbcDialect getDialect() {
        return new SqlserverDialect(); 
    }

What you expected to happen

this function should init SqlserverDialect with jtds driver base on config. for example:

       new SqlserverDialect(
                        jdbcConf.isWithNoLock(),
                        jdbcConf.getJdbcUrl().startsWith("jdbc:jtds:sqlserver"));

How to reproduce

just use a sql config for sqlserver

Anything else

No response

Version

1.12_release

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

Contributor Guide