🐛 Source LinkedIn Ads: fixed issue with normalization on redshift

[bazarnov]

What

Resolving: #13018

How

• renamed pivot key in Analytics streams, to be _pivot to avoid name conflicts on Redshift, since pivot is the reserved keyword.
• re-factored the public documentation to reflect standard_template
• increased unit_test coverage to 92%
• labeled source-linkedin-ads as generally_available for UI

🚨 User Impact 🚨

Customers should Refresh source schema for all existing connections, after the update to 0.1.8 since the schema should also reflect the updated key name. Or do not update if your existing connection is working properly and your destination is different from Redshift.

Pre-merge Checklist

Expand the relevant checklist and delete the others.

Updating a connector

Community member or Airbyter

• Unit & integration tests added and passing. Community members, please provide proof of success locally e.g: screenshot or copy-paste unit, integration, and acceptance test output. To run acceptance tests for a Python connector, follow instructions in the README. For java connectors run ./gradlew :airbyte-integrations:connectors:<name>:integrationTest.
• Code reviews completed
• Documentation updated
  • Changelog updated in docs/integrations/<source or destination>/<name>.md including changelog. See changelog example
• PR name follows PR naming conventions

Airbyter

If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.

• Create a non-forked branch based on this PR and test the below items on it
• Build is successful
• If new credentials are required for use in CI, add them to GSM. Instructions.
• /test connector=connectors/<name> command is passing
• New Connector version released on Dockerhub and connector version bumped by running the /publish command described here

[bazarnov]

/test connector=connectors/source-linkedin-ads

🕑 connectors/source-linkedin-ads https://github.com/airbytehq/airbyte/actions/runs/2454297657
✅ connectors/source-linkedin-ads https://github.com/airbytehq/airbyte/actions/runs/2454297657
Python tests coverage:

Name                                                 Stmts   Miss  Cover
------------------------------------------------------------------------
source_acceptance_test/utils/__init__.py                 6      0   100%
source_acceptance_test/tests/__init__.py                 4      0   100%
source_acceptance_test/__init__.py                       2      0   100%
source_acceptance_test/tests/test_full_refresh.py       52      2    96%
source_acceptance_test/utils/asserts.py                 37      2    95%
source_acceptance_test/config.py                        77      6    92%
source_acceptance_test/utils/json_schema_helper.py     105     13    88%
source_acceptance_test/tests/test_incremental.py       121     25    79%
source_acceptance_test/utils/common.py                  80     17    79%
source_acceptance_test/tests/test_core.py              294    106    64%
source_acceptance_test/utils/compare.py                 62     23    63%
source_acceptance_test/base.py                          10      4    60%
source_acceptance_test/utils/connector_runner.py       110     48    56%
------------------------------------------------------------------------
TOTAL                                                  960    246    74%
Name                               Stmts   Miss  Cover
------------------------------------------------------
source_linkedin_ads/analytics.py      45      0   100%
source_linkedin_ads/__init__.py        2      0   100%
source_linkedin_ads/utils.py          90      5    94%
source_linkedin_ads/source.py        168     20    88%
------------------------------------------------------
TOTAL                                305     25    92%

Build Passed

Test summary info:

All Passed

[lazebnyi]

LGTM. Few small comments.

[bazarnov]

/publish connector=connectors/source-linkedin-ads

🕑 connectors/source-linkedin-ads https://github.com/airbytehq/airbyte/actions/runs/2462165676
🚀 Successfully published connectors/source-linkedin-ads
🚀 Auto-bumped version for connectors/source-linkedin-ads
✅ connectors/source-linkedin-ads https://github.com/airbytehq/airbyte/actions/runs/2462165676