Skip to content

HIVE-26170: Code cleanup in jdbc dataconnector#3237

Merged
nrg4878 merged 1 commit intoapache:masterfrom
zhangbutao:clueanup_jdbc_dataconnector
Apr 26, 2022
Merged

HIVE-26170: Code cleanup in jdbc dataconnector#3237
nrg4878 merged 1 commit intoapache:masterfrom
zhangbutao:clueanup_jdbc_dataconnector

Conversation

@zhangbutao
Copy link
Contributor

What changes were proposed in this pull request?

Clean up unused import;

Fix incorrect Logger in PostgreSQLConnectorProvider;

Why are the changes needed?

Code cleanup

Does this PR introduce any user-facing change?

No

How was this patch tested?

No need tests

@zhangbutao
Copy link
Contributor Author

@nrg4878 @kgyrtkirk Would you mind taking a look? thx

Copy link
Contributor

@nrg4878 nrg4878 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

@nrg4878 nrg4878 merged commit a6ef9dc into apache:master Apr 26, 2022
DongWei-4 pushed a commit to DongWei-4/hive that referenced this pull request Oct 28, 2022
dengzhhu653 pushed a commit to dengzhhu653/hive that referenced this pull request Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants