ADAP-1125: Migrate dbt-athena into dbt-adapters

[mikealfare]

Migrate dbt-athena into this repo using the following shell script and call:

repo=$1
source_branch=$2
target_branch=$3

# create a remote for the legacy adapter repo and fetch the latest commits
git remote remove old || true
git remote add old https://github.com/dbt-labs/$repo.git
git fetch old

# merge the updated branch from the legacy repo into the dbt-adapters repo
git checkout $target_branch
git merge old/$source_branch --allow-unrelated-histories

# remove the remote that was created by this process
git remote remove old || true

source ./scripts/migrate-adapter.sh dbt-athena main monorepo/migrate-dbt-athena

Update workflows to include dbt-athena and dbt-athena-community.

Make updates related with code quality checks due to slightly different config.

Merge this PR as a regular merge instead of a squash merge in order to retain git blame history.

[cla-bot]

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @maiarareinaldo, @sashakcc, @keshihoriuchi, @voroninman, @lukas-gust, @nicor88, @skirino, @brunofaustino, @svdimchenko, @artem-garmash, @hiro-o918

[mikealfare]

The CLA is failing because this PR contains all commits ever made to dbt-athena, which includes those prior to dbt Labs assuming ownership.

I ran all of the PR checks by adding the pull_request trigger to pull_request_checks.yml. The results passed. I then removed the trigger since we only want pull_request_target as a trigger.

[cla-bot]

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @maiarareinaldo, @sashakcc, @keshihoriuchi, @voroninman, @lukas-gust, @nicor88, @skirino, @brunofaustino, @svdimchenko, @artem-garmash, @hiro-o918, @SoumayaMauthoorMOJ

[cla-bot]

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @maiarareinaldo, @sashakcc, @keshihoriuchi, @voroninman, @lukas-gust, @nicor88, @skirino, @brunofaustino, @svdimchenko, @artem-garmash, @hiro-o918, @SoumayaMauthoorMOJ

[cla-bot]

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @maiarareinaldo, @sashakcc, @keshihoriuchi, @voroninman, @lukas-gust, @nicor88, @skirino, @brunofaustino, @svdimchenko, @artem-garmash, @hiro-o918, @SoumayaMauthoorMOJ

[VersusFacit]

Two comments, probably minor. Nice to see unit tests are passing. I'd love to see a functional test run pass too. Not sure if that's possible at this stage. It would make sense to also have that follow on after merging and hotfixing as needed I suppose...

[mikealfare]

Two comments, probably minor. Nice to see unit tests are passing. I'd love to see a functional test run pass too. Not sure if that's possible at this stage. It would make sense to also have that follow on after merging and hotfixing as needed I suppose...

It got pushed up off the screen. See my post above.

[VersusFacit]

lets go!