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

[Bug]: Windows file escape for checksum #303

Closed
frannydelaney opened this issue May 3, 2023 · 1 comment · Fixed by #353
Closed

[Bug]: Windows file escape for checksum #303

frannydelaney opened this issue May 3, 2023 · 1 comment · Fixed by #353
Labels
bug Something isn't working

Comments

@frannydelaney
Copy link

Overview

Our Windows users are receiving errors when running dbt_artifacts due to how dbt_core handles large seed files (information found here on currently handling). When checksum is just stored as a file path, they encounter the same error as was reported here and resolved here

Error message:

14:56:11
14:56:11  Running 3 on-run-end hooks
14:56:11  Uploading model executions
14:56:11  Uploading seed executions
14:56:13  Uploading snapshot executions
14:56:13  Uploading test executions
14:56:13  Uploading exposures
14:56:13  Uploading tests
14:56:22  Uploading seeds
14:56:22  Database error while running on-run-end
14:56:23  Encountered an error:
Database Error
  001831 (42601): SQL compilation error: error line 2,019 at position 16
  Invalid unicode escape sequence '\u'; should be exactly 4 digits.

I haven't contributed to the project but would be happy to give it a shot if it's as simple as the previously noted change.

How to reproduce

Run dbt_artifacts on Windows machine where at least one seeds exceeds 1 MG.

Expected behaviour

Escape the checksum column for this situation as with the original_file_path escape added here

Screenshots

Environment

Results of running dbt --version:

Core:
  - installed: 1.3.1
  - latest:    1.5.0 - Update available!

  Your version of dbt-core is out of date!
  You can find instructions for upgrading here:
  https://docs.getdbt.com/docs/installation

Please paste the contents of your packages.yml file here:

packages:
  - package: brooklyn-data/dbt_artifacts
    version: 2.2.2
  - package: calogica/dbt_date
    version: 0.5.7
  - package: calogica/dbt_expectations
    version: 0.5.6
  - package: dbt-labs/dbt_utils
    version: 0.8.6
  - package: elementary-data/elementary
    version: 0.7.3
    ## Docs: https://docs.elementary-data.com
  - package: re-data/re_data
    version: 0.10.6
@frannydelaney frannydelaney added the bug Something isn't working label May 3, 2023
@glsdown
Copy link
Contributor

glsdown commented May 5, 2023

HI @frannydelaney thanks for reaching out.

If you are happy to get involved in resolving this, that would be great - thank you! Check out our contributing guide on how to get started. Feel free to reach out as well if you need any support with getting started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants