[Source-postgres] Set default cursor value for cdc mode #27442

nguyenaiden · 2023-06-16T18:24:13Z

What

26807
To prepare for Destination v2, Cdc-enabled DB sources need to have a pre-defined cursor for normalization to operate properly.

How

During the discover phase, source_defined_cursor is set to true, and _ab_cdc_lsn is selected as the predefined cursor. NOTE: This value could change if we believe it's not unique.

AirbyteStateMessage:

{
  "cdc": true,
  "cdc_state": {
    "state": {
      "[\"postgres\",{\"server\":\"postgres\"}]": "{\"transaction_id\":null,\"lsn\":840386496,\"txId\":1895971,\"ts_usec\":1686867930174234}"
    }
  },
  "streams": [
    {
      "stream_name": "blah",
      "stream_namespace": "public",
      "cursor_field": [
        "_ab_cdc_lsn"
      ]
    },
    {
      "stream_name": "jsonb_test_norm",
      "stream_namespace": "public",
      "cursor_field": [
        "_ab_cdc_lsn"
      ]
    }
  ]
}

🚨 User Impact 🚨

New CDC syncs that have refreshed their source schema will see this cursor field be chosen. Customers who reset their data will continue to see old behavior until their source goes through the discover() phase again.

github-actions · 2023-06-16T18:24:35Z

Before Merging a Connector Pull Request

Wow! What a great pull request you have here! 🎉

To merge this PR, ensure the following has been done/considered for each connector added or updated:

PR name follows PR naming conventions
Breaking changes are considered. If a Breaking Change is being introduced, ensure an Airbyte engineer has created a Breaking Change Plan and you've followed all steps in the Breaking Changes Checklist
Connector version has been incremented in the Dockerfile and metadata.yaml according to our Semantic Versioning for Connectors guidelines
Secrets in the connector's spec are annotated with airbyte_secret
All documentation files are up to date. (README.md, bootstrap.md, docs.md, etc...)
Changelog updated in docs/integrations/<source or destination>/<name>.md with an entry for the new version. See changelog example
The connector tests are passing in CI
You've updated the connector's metadata.yaml file (new!)
If set, you've ensured the icon is present in the platform-internal repo. (Docs)

If the checklist is complete, but the CI check is failing,

Check for hidden checklists in your PR description
Toggle the github label checklist-action-run on/off to re-run the checklist CI.

github-actions · 2023-06-16T18:29:31Z

Affected Connector Report

NOTE ⚠️ Changes in this PR affect the following connectors. Make sure to do the following as needed:

Run integration tests
Bump connector or module version
Add changelog
Publish the new version

✅ Sources (3)

Connector	Version	Changelog	Publish
`source-alloydb`	`2.0.28`	✅	✅
`source-alloydb-strict-encrypt`	`2.0.28`	🔵 (ignored)	🔵 (ignored)
`source-postgres-strict-encrypt`	`2.0.33`	🔵 (ignored)	🔵 (ignored)

See "Actionable Items" below for how to resolve warnings and errors.

✅ Destinations (0)

Connector	Version	Changelog	Publish

See "Actionable Items" below for how to resolve warnings and errors.

✅ Other Modules (0)

Actionable Items

(click to expand)

Category	Status	Actionable Item
Version	❌ mismatch	The version of the connector is different from its normal variant. Please bump the version of the connector.
	⚠ doc not found	The connector does not seem to have a documentation file. This can be normal (e.g. basic connector like `source-jdbc` is not published or documented). Please double-check to make sure that it is not a bug.
Changelog	⚠ doc not found	The connector does not seem to have a documentation file. This can be normal (e.g. basic connector like `source-jdbc` is not published or documented). Please double-check to make sure that it is not a bug.
	❌ changelog missing	There is no chnagelog for the current version of the connector. If you are the author of the current version, please add a changelog.
Publish	⚠ not in seed	The connector is not in the cloud or oss registry, so its publication status cannot be checked. This can be normal (e.g. some connectors are cloud-specific, and only listed in the cloud seed file). Please double-check to make sure that you have added a metadata.yaml file and the expected registries are enabled.

octavia-squidington-iii · 2023-06-16T18:31:40Z

source-alloydb-strict-encrypt test report (commit `fa057b48c9`) - ❌

⏲️ Total pipeline duration: 369 seconds

Step	Result
Validate airbyte-integrations/connectors/source-alloydb-strict-encrypt/metadata.yaml	✅
Connector version semver check.	✅
Connector version increment check.	❌
QA checks	✅
Build connector tar	✅
Build source-alloydb-strict-encrypt docker image for platform linux/x86_64	✅
Unit tests	✅
Integration tests	✅

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-alloydb-strict-encrypt test

github-actions · 2023-06-16T18:38:10Z

Coverage report for source-postgres

File	Coverage [89.6%]	🍏
PostgresCatalogHelper.java	100%	🍏
PostgresSource.java	88.5%	🍏

Total Project Coverage	69.94%	🍏

octavia-squidington-iii · 2023-06-16T18:41:57Z

source-postgres-strict-encrypt test report (commit `fa057b48c9`) - ❌

⏲️ Total pipeline duration: 610 seconds

Step	Result
Validate airbyte-integrations/connectors/source-postgres-strict-encrypt/metadata.yaml	✅
Connector version semver check.	✅
Connector version increment check.	❌
QA checks	✅
Build connector tar	✅
Build source-postgres-strict-encrypt docker image for platform linux/x86_64	✅
Unit tests	✅
Integration tests	✅

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-postgres-strict-encrypt test

octavia-squidington-iii · 2023-06-16T19:18:06Z

source-mssql test report (commit `fa057b48c9`) - ❌

⏲️ Total pipeline duration: 2163 seconds

Step	Result
Validate airbyte-integrations/connectors/source-mssql/metadata.yaml	✅
Connector version semver check.	✅
Connector version increment check.	❌
QA checks	✅
Build connector tar	✅
Build source-mssql docker image for platform linux/x86_64	✅
Unit tests	✅
Integration tests	❌

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-mssql test

octavia-squidington-iii · 2023-06-16T19:23:33Z

source-alloydb test report (commit `fa057b48c9`) - ❌

⏲️ Total pipeline duration: 321 seconds

Step	Result
Validate airbyte-integrations/connectors/source-alloydb/metadata.yaml	✅
Connector version semver check.	✅
Connector version increment check.	❌
QA checks	✅
Build connector tar	✅
Build source-alloydb docker image for platform linux/x86_64	✅
Unit tests	✅
Integration tests	✅

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-alloydb test

octavia-squidington-iii · 2023-06-17T18:31:12Z

source-alloydb-strict-encrypt test report (commit `48b836dd39`) - ❌

⏲️ Total pipeline duration: 325 seconds

Step	Result
Validate airbyte-integrations/connectors/source-alloydb-strict-encrypt/metadata.yaml	✅
Connector version semver check.	✅
Connector version increment check.	❌
QA checks	✅
Build connector tar	✅
Build source-alloydb-strict-encrypt docker image for platform linux/x86_64	✅
Unit tests	✅
Integration tests	✅

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-alloydb-strict-encrypt test

octavia-squidington-iii · 2023-06-17T19:07:20Z

source-mssql test report (commit `48b836dd39`) - ❌

⏲️ Total pipeline duration: 2162 seconds

Step	Result
Validate airbyte-integrations/connectors/source-mssql/metadata.yaml	✅
Connector version semver check.	❌
Connector version increment check.	❌
QA checks	✅
Build connector tar	✅
Build source-mssql docker image for platform linux/x86_64	✅
Unit tests	✅
Integration tests	❌

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-mssql test

octavia-squidington-iii · 2023-06-17T19:37:22Z

source-mysql-strict-encrypt test report (commit `48b836dd39`) - ❌

⏲️ Total pipeline duration: 1796 seconds

Step	Result
Validate airbyte-integrations/connectors/source-mysql-strict-encrypt/metadata.yaml	✅
Connector version semver check.	✅
Connector version increment check.	❌
QA checks	✅
Build connector tar	✅
Build source-mysql-strict-encrypt docker image for platform linux/x86_64	✅
Unit tests	❌

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-mysql-strict-encrypt test

octavia-squidington-iii · 2023-06-17T19:58:15Z

source-postgres test report (commit `48b836dd39`) - ❌

⏲️ Total pipeline duration: 1247 seconds

Step	Result
Validate airbyte-integrations/connectors/source-postgres/metadata.yaml	✅
Connector version semver check.	✅
Connector version increment check.	❌
QA checks	✅
Build connector tar	✅
Build source-postgres docker image for platform linux/x86_64	✅
Unit tests	✅
Integration tests	✅

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-postgres test

prateekmukhedkar

So just to confirm, _ab_cdc_lsn field and associated value is already present in the data that we serialize, and that is why we don't need to add this field as we reading data?

nguyenaiden · 2023-06-21T23:03:10Z

@prateekmukhedkar yes, we are currently emitting the lsn value in the data we serialize so we don't need to add a new column.

edgao · 2023-06-23T18:00:52Z

running legacy normalization tests in #27670 (comment). Will approve once those look good.

edgao

tests are green on #27670.

thanks for putting this together so quickly!

subodh1810

THis looks fine. I just want you to carry out few tests

Create a postgres source cdc connector on an old version with a destination that supports normalization (snowflake), run a sync and then change image version of the connector to use changes from this PR and then run a sync and make sure everything works.
Do everything as mentioned in step 1 but this time refresh the schema as well.
Create a connector on this new version and then roll back to the old version and see what happens (both with and without refresh schema)

octavia-squidington-iii · 2023-07-12T18:25:08Z

source-postgres test report (commit `052a5aee9d`) - ❌

⏲️ Total pipeline duration: 10mn18s

Step	Result
Validate airbyte-integrations/connectors/source-postgres/metadata.yaml	✅
Connector version semver check	✅
QA checks	✅
Build connector tar	✅
Build source-postgres docker image for platform linux/x86_64	✅
./gradlew :airbyte-integrations:connectors:source-postgres:integrationTest	✅
Acceptance tests	❌

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-postgres test

octavia-squidington-iii · 2023-07-12T18:26:47Z

source-mysql test report (commit `052a5aee9d`) - ❌

⏲️ Total pipeline duration: 01mn29s

Step	Result
Validate airbyte-integrations/connectors/source-mysql/metadata.yaml	✅
Connector version semver check	✅
QA checks	✅
Build connector tar	✅
Build source-mysql docker image for platform linux/x86_64	✅
./gradlew :airbyte-integrations:connectors:source-mysql:integrationTest	❌
Acceptance tests	✅

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-mysql test

octavia-squidington-iii · 2023-07-12T18:32:39Z

source-postgres-strict-encrypt test report (commit `052a5aee9d`) - ❌

⏲️ Total pipeline duration: 05mn43s

Step	Result
Validate airbyte-integrations/connectors/source-postgres-strict-encrypt/metadata.yaml	✅
Connector version semver check	✅
QA checks	✅
Build connector tar	✅
Build source-postgres-strict-encrypt docker image for platform linux/x86_64	✅
./gradlew :airbyte-integrations:connectors:source-postgres-strict-encrypt:integrationTest	✅
Acceptance tests	❌

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-postgres-strict-encrypt test

octavia-squidington-iii · 2023-07-12T18:45:19Z

source-mysql-strict-encrypt test report (commit `052a5aee9d`) - ❌

⏲️ Total pipeline duration: 12mn30s

Step	Result
Validate airbyte-integrations/connectors/source-mysql-strict-encrypt/metadata.yaml	✅
Connector version semver check	✅
QA checks	✅
Build connector tar	✅
Build source-mysql-strict-encrypt docker image for platform linux/x86_64	✅
./gradlew :airbyte-integrations:connectors:source-mysql-strict-encrypt:integrationTest	❌
Acceptance tests	❌

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-mysql-strict-encrypt test

octavia-squidington-iii · 2023-07-12T18:49:49Z

source-alloydb-strict-encrypt test report (commit `052a5aee9d`) - ❌

⏲️ Total pipeline duration: 04mn18s

Step	Result
Validate airbyte-integrations/connectors/source-alloydb-strict-encrypt/metadata.yaml	✅
Connector version semver check	✅
QA checks	✅
Build connector tar	✅
Build source-alloydb-strict-encrypt docker image for platform linux/x86_64	✅
./gradlew :airbyte-integrations:connectors:source-alloydb-strict-encrypt:integrationTest	✅
Acceptance tests	❌

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-alloydb-strict-encrypt test

…a breaking change

nguyenaiden · 2023-07-12T20:38:51Z

/legacy-test connector=connectors/source-postgres

🕑 connectors/source-postgres https://github.com/airbytehq/airbyte/actions/runs/5536228889
✅ connectors/source-postgres https://github.com/airbytehq/airbyte/actions/runs/5536228889
No Python unittests run

Build Passed

Test summary info:

=========================== short test summary info ============================
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/plugin.py:63: Skipping TestIncremental.test_two_sequential_reads: not found in the config.
SKIPPED [2] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:100: The previous and actual specifications are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:615: The previous and actual discovered catalogs are identical.
SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:624: Backward compatibility tests are disabled for version 2.1.1.
SKIPPED [2] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:712: This tests currently leads to too much failures. We need to fix the connectors at scale first.
=================== 70 passed, 7 skipped in 83.48s (0:01:23) ===================

nguyenaiden · 2023-07-12T20:42:52Z

/legacy-test connector=connectors/source-postgres-strict-encrypt

🕑 connectors/source-postgres-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/5536256927
✅ connectors/source-postgres-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/5536256927
No Python unittests run

Build Passed

Test summary info:

All Passed

nguyenaiden · 2023-07-12T20:43:33Z

/legacy-test connector=connectors/source-alloydb

🕑 connectors/source-alloydb https://github.com/airbytehq/airbyte/actions/runs/5536261878
✅ connectors/source-alloydb https://github.com/airbytehq/airbyte/actions/runs/5536261878
No Python unittests run

Build Passed

Test summary info:

All Passed

nguyenaiden · 2023-07-12T20:43:43Z

/test connector=connectors/source-alloydb-strict-encrypt

⚠️ The test slash command is now deprecated.

The connector tests are automatically triggered as CI checks.

Please use /legacy-test if you need to test CDK or CAT changes.

Please join post to #pipeline-complaint-hotline slack channel if something is not working as expected.

nguyenaiden · 2023-07-12T21:30:11Z

/legacy-test connector=connectors/source-alloydb-strict-encrypt

🕑 connectors/source-alloydb-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/5536628041
✅ connectors/source-alloydb-strict-encrypt https://github.com/airbytehq/airbyte/actions/runs/5536628041
No Python unittests run

Build Passed

Test summary info:

All Passed

pedroslopez

Didn't review much on the tests side, leaving that to the sources team.

From ops side looks good, seeing breaking changes checks correctly indicated as skipped in the test logs for the affected version:

SKIPPED [1] ../usr/local/lib/python3.9/site-packages/connector_acceptance_test/tests/test_core.py:624: Backward compatibility tests are disabled for version 2.1.1.

Should make sure acceptance tests are passing for the connectors to be published - it looks like they are.

Build failure for destination-snowflake is also failing on master and this PR does not touch it, so should be ok to release.

pedroslopez · 2023-07-12T21:06:08Z

...rs/source-postgres/src/main/java/io/airbyte/integrations/source/postgres/PostgresSource.java

          .map(PostgresCatalogHelper::setIncrementalToSourceDefined)
+          .map(PostgresCatalogHelper::setDefaultCursorFieldForCdc)


👍🏼 only for cdc, we are setting the default cursor.

nguyenaiden · 2023-07-13T16:12:01Z

/approve-and-merge reason="All CI steps already passed. And manual tests done to verify. Author: Duy Nguyen, copilot=Ed Gao"

octavia-approvington · 2023-07-13T16:12:32Z

Our work here is done

octavia-squidington-iii · 2023-07-13T19:18:18Z

source-mysql-strict-encrypt test report (commit `5c2e4796ab`) - ❌

⏲️ Total pipeline duration: 12mn58s

Step	Result
Validate airbyte-integrations/connectors/source-mysql-strict-encrypt/metadata.yaml	✅
Connector version semver check	✅
QA checks	✅
Build connector tar	✅
Build source-mysql-strict-encrypt docker image for platform linux/x86_64	✅
./gradlew :airbyte-integrations:connectors:source-mysql-strict-encrypt:integrationTest	❌
Acceptance tests	❌

🔗 View the logs here

Please note that tests are only run on PR ready for review. Please set your PR to draft mode to not flood the CI engine and upstream service on following commits.
You can run the same pipeline locally on this branch with the airbyte-ci tool with the following command

airbyte-ci connectors --name=source-mysql-strict-encrypt test

) * use LSN as default cursor for postgres CDC * Fixed static constant * Set lsn default cursor value for postgres sync * Bumped metadata and dockerfile versions * Disable acceptance backwards compatibility discovery test as this is a breaking change --------- Co-authored-by: Conor <cpdeethree@users.noreply.github.com>

nguyenaiden marked this pull request as ready for review June 16, 2023 18:24

nguyenaiden requested a review from a team as a code owner June 16, 2023 18:24

octavia-squidington-iii added area/connectors Connector related issues connectors/source/mssql connectors/source/mysql connectors/source/postgres labels Jun 16, 2023

nguyenaiden requested review from edgao, subodh1810, akashkulk and prateekmukhedkar June 16, 2023 18:24

prateekmukhedkar reviewed Jun 20, 2023

View reviewed changes

edgao mentioned this pull request Jun 23, 2023

Normalization integration tests: set explicit cursor on cdc streams #27670

Merged

edgao approved these changes Jun 23, 2023

View reviewed changes

subodh1810 reviewed Jun 26, 2023

View reviewed changes

prateekmukhedkar added the breaking-change Don't merge me unless you are ready. label Jul 12, 2023

prateekmukhedkar mentioned this pull request Jul 12, 2023

source-mssql: Duplicate rows for same LSN #24206

Closed

nguyenaiden force-pushed the duy/cdc-source-defined-cursor branch from 48b836d to 052a5ae Compare July 12, 2023 18:13

octavia-squidington-iii added the connectors/source/postgres-strict-encrypt label Jul 12, 2023

nguyenaiden added 5 commits July 12, 2023 12:05

use LSN as default cursor for postgres CDC

69cb66f

Fixed static constant

1fe30e2

Set lsn default cursor value for postgres sync

64569cf

Bumped metadata and dockerfile versions

e5cc6b7

Disable acceptance backwards compatibility discovery test as this is …

54f4a11

…a breaking change

nguyenaiden force-pushed the duy/cdc-source-defined-cursor branch from b661b1f to 54f4a11 Compare July 12, 2023 19:05

Merge branch 'master' into duy/cdc-source-defined-cursor

ebeb7eb

nguyenaiden mentioned this pull request Jul 12, 2023

[Source-Postgres] Rollback to 2.1.1 in case 3.0.0 fails [DO NOT MERGE] #28251

Closed

pedroslopez approved these changes Jul 12, 2023

View reviewed changes

Duy Nguyen added 2 commits July 13, 2023 07:49

Merge branch 'master' into duy/cdc-source-defined-cursor

bf79524

Merge branch 'master' into duy/cdc-source-defined-cursor

c07e671

octavia-approvington approved these changes Jul 13, 2023

View reviewed changes

octavia-approvington merged commit 633c939 into master Jul 13, 2023

octavia-approvington deleted the duy/cdc-source-defined-cursor branch July 13, 2023 16:12

		.map(PostgresCatalogHelper::setIncrementalToSourceDefined)
		.map(PostgresCatalogHelper::setDefaultCursorFieldForCdc)

[Source-postgres] Set default cursor value for cdc mode #27442

[Source-postgres] Set default cursor value for cdc mode #27442

Conversation

nguyenaiden commented Jun 16, 2023 • edited Loading

What

How

Recommended reading order

🚨 User Impact 🚨

github-actions bot commented Jun 16, 2023 • edited by nguyenaiden Loading

Before Merging a Connector Pull Request

github-actions bot commented Jun 16, 2023 • edited Loading

Affected Connector Report

✅ Sources (3)

✅ Destinations (0)

✅ Other Modules (0)

Actionable Items

octavia-squidington-iii commented Jun 16, 2023

source-alloydb-strict-encrypt test report (commit fa057b48c9) - ❌

github-actions bot commented Jun 16, 2023 • edited Loading

Coverage report for source-postgres

octavia-squidington-iii commented Jun 16, 2023

source-postgres-strict-encrypt test report (commit fa057b48c9) - ❌

octavia-squidington-iii commented Jun 16, 2023

source-mssql test report (commit fa057b48c9) - ❌

octavia-squidington-iii commented Jun 16, 2023

source-alloydb test report (commit fa057b48c9) - ❌

octavia-squidington-iii commented Jun 17, 2023

source-alloydb-strict-encrypt test report (commit 48b836dd39) - ❌

octavia-squidington-iii commented Jun 17, 2023

source-mssql test report (commit 48b836dd39) - ❌

octavia-squidington-iii commented Jun 17, 2023

source-mysql-strict-encrypt test report (commit 48b836dd39) - ❌

octavia-squidington-iii commented Jun 17, 2023

source-postgres test report (commit 48b836dd39) - ❌

prateekmukhedkar left a comment

Choose a reason for hiding this comment

nguyenaiden commented Jun 21, 2023

edgao commented Jun 23, 2023

edgao left a comment

Choose a reason for hiding this comment

subodh1810 left a comment

Choose a reason for hiding this comment

octavia-squidington-iii commented Jul 12, 2023

source-postgres test report (commit 052a5aee9d) - ❌

octavia-squidington-iii commented Jul 12, 2023

source-mysql test report (commit 052a5aee9d) - ❌

octavia-squidington-iii commented Jul 12, 2023

source-postgres-strict-encrypt test report (commit 052a5aee9d) - ❌

octavia-squidington-iii commented Jul 12, 2023

source-mysql-strict-encrypt test report (commit 052a5aee9d) - ❌

octavia-squidington-iii commented Jul 12, 2023

source-alloydb-strict-encrypt test report (commit 052a5aee9d) - ❌

nguyenaiden commented Jul 12, 2023 • edited by github-actions bot Loading

Build Passed

nguyenaiden commented Jul 12, 2023 • edited by github-actions bot Loading

Build Passed

nguyenaiden commented Jul 12, 2023 • edited by github-actions bot Loading

Build Passed

nguyenaiden commented Jul 12, 2023 • edited by github-actions bot Loading

nguyenaiden commented Jul 12, 2023 • edited by github-actions bot Loading

Build Passed

pedroslopez left a comment • edited Loading

Choose a reason for hiding this comment

pedroslopez Jul 12, 2023

Choose a reason for hiding this comment

nguyenaiden commented Jul 13, 2023

octavia-approvington commented Jul 13, 2023

octavia-squidington-iii commented Jul 13, 2023

source-mysql-strict-encrypt test report (commit 5c2e4796ab) - ❌

nguyenaiden commented Jun 16, 2023 •

edited

Loading

github-actions bot commented Jun 16, 2023 •

edited by nguyenaiden

Loading

github-actions bot commented Jun 16, 2023 •

edited

Loading

source-alloydb-strict-encrypt test report (commit `fa057b48c9`) - ❌

github-actions bot commented Jun 16, 2023 •

edited

Loading

source-postgres-strict-encrypt test report (commit `fa057b48c9`) - ❌

source-mssql test report (commit `fa057b48c9`) - ❌

source-alloydb test report (commit `fa057b48c9`) - ❌

source-alloydb-strict-encrypt test report (commit `48b836dd39`) - ❌

source-mssql test report (commit `48b836dd39`) - ❌

source-mysql-strict-encrypt test report (commit `48b836dd39`) - ❌

source-postgres test report (commit `48b836dd39`) - ❌

source-postgres test report (commit `052a5aee9d`) - ❌

source-mysql test report (commit `052a5aee9d`) - ❌

source-postgres-strict-encrypt test report (commit `052a5aee9d`) - ❌

source-mysql-strict-encrypt test report (commit `052a5aee9d`) - ❌

source-alloydb-strict-encrypt test report (commit `052a5aee9d`) - ❌

nguyenaiden commented Jul 12, 2023 •

edited by github-actions bot

Loading

nguyenaiden commented Jul 12, 2023 •

edited by github-actions bot

Loading

nguyenaiden commented Jul 12, 2023 •

edited by github-actions bot

Loading

nguyenaiden commented Jul 12, 2023 •

edited by github-actions bot

Loading

nguyenaiden commented Jul 12, 2023 •

edited by github-actions bot

Loading

pedroslopez left a comment •

edited

Loading

source-mysql-strict-encrypt test report (commit `5c2e4796ab`) - ❌