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

Revert #367 #378

Merged
merged 1 commit into from
Jun 30, 2022
Merged

Revert #367 #378

merged 1 commit into from
Jun 30, 2022

Conversation

jtcohen6
Copy link
Contributor

@jtcohen6 jtcohen6 commented Jun 29, 2022

#367 added an environment variable to server-side user-agent tracking. This was a mistake, for two reasons:

  • We are moving to standardize and simplify the information that dbt supplies in user agents. This should be only dbt, or dbt-{version} where appropriate. The addition of this env var would make the user agent much less consistent.
  • The environment variable I included is an internal mechanism, not a reliable external contract to which we've committed. It could change in the future without warning. It isn't the right way to share this contextual information.

@jtcohen6 jtcohen6 requested a review from dataders June 29, 2022 10:22
@cla-bot cla-bot bot added the cla:yes label Jun 29, 2022
- Initialize lift + shift for cross-db macros ([#359](https://github.com/dbt-labs/dbt-spark/pull/359))
- Add invocation env to user agent string ([#367](https://github.com/dbt-labs/dbt-spark/pull/367))
- Use dispatch pattern for get_columns_in_relation_raw macro ([#365](https://github.com/dbt-labs/dbt-spark/pull/365))

### Contributors
- [@ueshin](https://github.com/dbt-labs/dbt-spark/commits?author=ueshin) ([#365](https://github.com/dbt-labs/dbt-spark/pull/365))
- [@ueshin](https://github.com/ueshin) ([#365](https://github.com/dbt-labs/dbt-spark/pull/365))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I just happened to notice this while resolving changelog conflicts. Unrelated to the substantive changes being reverted in this PR.

Copy link
Contributor

Choose a reason for hiding this comment

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

oops, thanks for the fix!

@jtcohen6
Copy link
Contributor Author

The failing test will be resolved in a separate PR: #375

@jtcohen6 jtcohen6 merged commit 48e1989 into main Jun 30, 2022
@jtcohen6 jtcohen6 deleted the revert-367 branch June 30, 2022 10:23
ueshin added a commit to databricks/dbt-databricks that referenced this pull request Jul 5, 2022
### Description

Reverts #106 to follow dbt-labs/dbt-spark#378.
ueshin added a commit to databricks/dbt-databricks that referenced this pull request Jul 5, 2022
### Description

Reverts #106 to follow dbt-labs/dbt-spark#378.
francescomucio pushed a commit to francescomucio/dbt-spark that referenced this pull request Jul 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants