DTStack/chunjun

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

Open

#1.087 aberto em 24 de jul. de 2022

Ver no GitHub
 (2 comments) (0 reactions) (1 assignee)Java (1.668 forks)batch import
buggood first issue

Métricas do repositório

Stars
 (3.752 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
O bug está no método getDialect() de SqlserverDynamicTableFactory.java. A correção envolve modificar o método getDialect para passar jdbcConf.isWithNoLock() e uma verificação para URL jtds ao construtor de SqlserverDialect. Revise a classe SqlserverDialect para entender seus parâmetros de construtor. Nenhum PR ou comentário vinculado fornece orientação adicional, mas o relator do problema indicou disposição para enviar um PR.
Pilha de tecnologia
java
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitJava
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75