Skip to content

destination-redshift: Heartbeats for bastion and other minor bugfixes - #33948

Merged
Gireesh Sreepathi (gisripa) merged 22 commits into
masterfrom
gireesh/redshift/ssh-testing
Jan 8, 2024
Merged

destination-redshift: Heartbeats for bastion and other minor bugfixes#33948
Gireesh Sreepathi (gisripa) merged 22 commits into
masterfrom
gireesh/redshift/ssh-testing

Conversation

@gisripa

@gisripa Gireesh Sreepathi (gisripa) commented Jan 4, 2024

Copy link
Copy Markdown
Contributor

What

Fixes: #33945
Fixes: #33900

Potentially Fixes: #33643

How

See CDK changes #33727
Adding addition redshift connectTimeout from default 10s

Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
@vercel

vercel Bot commented Jan 4, 2024

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 6, 2024 0:08am

@octavia-squidington-iii Octavia Squidington III (octavia-squidington-iii) added the CDK Connector Development Kit label Jan 4, 2024
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
@gisripa Gireesh Sreepathi (gisripa) changed the title Gireesh/redshift/ssh testing destination-redshift: Heartbeats for bastion and other minor bugfixes Jan 5, 2024
@gisripa Gireesh Sreepathi (gisripa) removed the CDK Connector Development Kit label Jan 5, 2024

@edgao Edward Gao (edgao) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

maybe worth also releasing bigquery+snowflake for the T+D setup fix?

@gisripa

Copy link
Copy Markdown
Contributor Author

maybe worth also releasing bigquery+snowflake for the T+D setup fix?

Oh right. DefaultTD is a project dependency right ? so just version bumps should do ?

@edgao

Copy link
Copy Markdown
Contributor

just version bumps should do

yep

Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>

@edgao Edward Gao (edgao) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

two nits, otherwise lgtm!

…integration/resources/sqlgenerator/alltypes_expectedrecords_final.jsonl

Co-authored-by: Edward Gao <edward.gao@airbyte.io>
…java/io/airbyte/integrations/destination/redshift/RedshiftStagingS3Destination.java

Co-authored-by: Edward Gao <edward.gao@airbyte.io>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Signed-off-by: Gireesh Sreepathi <gisripa@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

3 participants