Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Destinations BigQuery & Snowflake: use latest base-java to emit AirbyteTraceMessage on errros #12805

Merged
merged 7 commits into from
May 12, 2022

Conversation

Phlair
Copy link
Contributor

@Phlair Phlair commented May 12, 2022

What

Follow-on from PR: #12614

This bumps our java connectors that are in GA to enable the AirbyteTraceMessage emission on errors.

@github-actions github-actions bot added area/connectors Connector related issues area/documentation Improvements or additions to documentation labels May 12, 2022
@Phlair
Copy link
Contributor Author

Phlair commented May 12, 2022

/test connector=connectors/destination-bigquery

🕑 connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/2312832506
✅ connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/2312832506
Python tests coverage:

Name                                                              Stmts   Miss  Cover
-------------------------------------------------------------------------------------
normalization/transform_config/__init__.py                            2      0   100%
normalization/transform_catalog/reserved_keywords.py                 13      0   100%
normalization/transform_catalog/__init__.py                           2      0   100%
normalization/destination_type.py                                    13      0   100%
normalization/__init__.py                                             4      0   100%
normalization/transform_catalog/destination_name_transformer.py     155      8    95%
normalization/transform_config/transform.py                         159     31    81%
normalization/transform_catalog/table_name_registry.py              174     34    80%
normalization/transform_catalog/utils.py                             34      7    79%
normalization/transform_catalog/dbt_macro.py                         22      7    68%
normalization/transform_catalog/catalog_processor.py                147     80    46%
normalization/transform_catalog/transform.py                         61     38    38%
normalization/transform_catalog/stream_processor.py                 534    345    35%
-------------------------------------------------------------------------------------
TOTAL                                                              1320    550    58%

@Phlair
Copy link
Contributor Author

Phlair commented May 12, 2022

/test connector=connectors/destination-bigquery-denormalized

🕑 connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/2312833204
✅ connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/2312833204
Python tests coverage:

Name                                                              Stmts   Miss  Cover
-------------------------------------------------------------------------------------
normalization/transform_config/__init__.py                            2      0   100%
normalization/transform_catalog/reserved_keywords.py                 13      0   100%
normalization/transform_catalog/__init__.py                           2      0   100%
normalization/destination_type.py                                    13      0   100%
normalization/__init__.py                                             4      0   100%
normalization/transform_catalog/destination_name_transformer.py     155      8    95%
normalization/transform_config/transform.py                         159     31    81%
normalization/transform_catalog/table_name_registry.py              174     34    80%
normalization/transform_catalog/utils.py                             34      7    79%
normalization/transform_catalog/dbt_macro.py                         22      7    68%
normalization/transform_catalog/catalog_processor.py                147     80    46%
normalization/transform_catalog/transform.py                         61     38    38%
normalization/transform_catalog/stream_processor.py                 534    345    35%
-------------------------------------------------------------------------------------
TOTAL                                                              1320    550    58%

@Phlair
Copy link
Contributor Author

Phlair commented May 12, 2022

/test connector=connectors/destination-snowflake

🕑 connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/2312833793
✅ connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/2312833793
Python tests coverage:

Name                                                              Stmts   Miss  Cover
-------------------------------------------------------------------------------------
normalization/transform_config/__init__.py                            2      0   100%
normalization/transform_catalog/reserved_keywords.py                 13      0   100%
normalization/transform_catalog/__init__.py                           2      0   100%
normalization/destination_type.py                                    13      0   100%
normalization/__init__.py                                             4      0   100%
normalization/transform_catalog/destination_name_transformer.py     155      8    95%
normalization/transform_config/transform.py                         159     31    81%
normalization/transform_catalog/table_name_registry.py              174     34    80%
normalization/transform_catalog/utils.py                             34      7    79%
normalization/transform_catalog/dbt_macro.py                         22      7    68%
normalization/transform_catalog/catalog_processor.py                147     80    46%
normalization/transform_catalog/transform.py                         61     38    38%
normalization/transform_catalog/stream_processor.py                 534    345    35%
-------------------------------------------------------------------------------------
TOTAL                                                              1320    550    58%

@Phlair
Copy link
Contributor Author

Phlair commented May 12, 2022

/publish connector=connectors/destination-bigquery

🕑 connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/2313067735
🚀 Successfully published connectors/destination-bigquery
🚀 Auto-bumped version for connectors/destination-bigquery
✅ connectors/destination-bigquery https://github.com/airbytehq/airbyte/actions/runs/2313067735

@Phlair
Copy link
Contributor Author

Phlair commented May 12, 2022

/publish connector=connectors/destination-bigquery-denormalized

🕑 connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/2313068755
🚀 Successfully published connectors/destination-bigquery-denormalized
🚀 Auto-bumped version for connectors/destination-bigquery-denormalized
✅ connectors/destination-bigquery-denormalized https://github.com/airbytehq/airbyte/actions/runs/2313068755

@Phlair
Copy link
Contributor Author

Phlair commented May 12, 2022

/publish connector=connectors/destination-snowflake

🕑 connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/2313069862
🚀 Successfully published connectors/destination-snowflake
🚀 Auto-bumped version for connectors/destination-snowflake
✅ connectors/destination-snowflake https://github.com/airbytehq/airbyte/actions/runs/2313069862

@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets May 12, 2022 12:00 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets May 12, 2022 12:00 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets May 12, 2022 12:03 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets May 12, 2022 12:04 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets May 12, 2022 12:13 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets May 12, 2022 12:13 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets May 12, 2022 13:11 Inactive
@octavia-squidington-iii octavia-squidington-iii temporarily deployed to more-secrets May 12, 2022 13:11 Inactive
@Phlair
Copy link
Contributor Author

Phlair commented May 12, 2022

merging this without review since it's just version bumping + publishing connectors, the relevant PR that has been reviewed is here: #12614

@Phlair Phlair merged commit 440f3db into master May 12, 2022
@Phlair Phlair deleted the george/ga-java-connectors-bump branch May 12, 2022 13:23
suhomud pushed a commit that referenced this pull request May 23, 2022
…teTraceMessage on errros (#12805)

* bumping BQ and Snowflake dests to get AirbyteTraceMessage from base-java

* add PR link to changelogs

* auto-bump connector version

* auto-bump connector version

* auto-bump connector version

Co-authored-by: Octavia Squidington III <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants