Skip to content

Update redshift jdbc to 2.1.0.20#19435

Merged
wendigo merged 1 commit intomasterfrom
serafin/redshift-jdbc42
Oct 18, 2023
Merged

Update redshift jdbc to 2.1.0.20#19435
wendigo merged 1 commit intomasterfrom
serafin/redshift-jdbc42

Conversation

@wendigo
Copy link
Copy Markdown
Contributor

@wendigo wendigo commented Oct 18, 2023

No release notes needed

@cla-bot cla-bot bot added the cla-signed label Oct 18, 2023
@wendigo wendigo requested a review from hashhar October 18, 2023 09:27
@hashhar
Copy link
Copy Markdown
Member

hashhar commented Oct 18, 2023

Please add a temp commit that enables ConnectorTest in surefire for CI's sake.

@wendigo
Copy link
Copy Markdown
Contributor Author

wendigo commented Oct 18, 2023

@hashhar can you elaborate? I can see that the redshift job will be executed

@wendigo wendigo merged commit d23921d into master Oct 18, 2023
@wendigo wendigo deleted the serafin/redshift-jdbc42 branch October 18, 2023 10:55
@github-actions github-actions bot added this to the 430 milestone Oct 18, 2023
@hashhar
Copy link
Copy Markdown
Member

hashhar commented Oct 18, 2023

@hashhar can you elaborate? I can see that the redshift job will be executed

See surefire config, Redshift's ConectorTest class is skipped by config.

@mosabua mosabua mentioned this pull request Oct 20, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hashhar hashhar Awaiting requested review from hashhar

Assignees

No one assigned

Labels

cla-signed

Milestone

430

Development

Successfully merging this pull request may close these issues.

2 participants

@wendigo @hashhar