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

chore: Upgrade xapi-db-load to 1.0 #550

Merged
merged 2 commits into from
Jan 18, 2024
Merged

chore: Upgrade xapi-db-load to 1.0 #550

merged 2 commits into from
Jan 18, 2024

Conversation

bmtcril
Copy link
Contributor

@bmtcril bmtcril commented Dec 20, 2023

Upgrades xapi-db-load to the new config-based version along with the tests and commands that use it.

@bmtcril bmtcril force-pushed the bmtcril/upgrade_db_load branch from 374059d to 0691648 Compare December 20, 2023 20:05
@bmtcril
Copy link
Contributor Author

bmtcril commented Dec 20, 2023

Hmm, tests all passed but there are actually errors in the logs for all 3 runs even though it worked locally. >_>

clickhouse_connect.driver.exceptions.DatabaseError: :HTTPDriver for http://clickhouse:8123 returned response code 404)
 Code: 60. DB::Exception: Table xapi.xapi_events_all_parsed does not exist. Maybe you meant xapi_events_all?. 

@bmtcril
Copy link
Contributor Author

bmtcril commented Dec 20, 2023

Oh, ha, dbt is creating everything in the reporting schema. That complicates things. cc @SoryRawyer

2 of 19 START sql materialized_view model reporting.xapi_events_all_parsed . [RUN]

@bmtcril bmtcril force-pushed the bmtcril/upgrade_db_load branch from 0691648 to f9b0e01 Compare January 16, 2024 18:54
@bmtcril
Copy link
Contributor Author

bmtcril commented Jan 16, 2024

@Ian2012 this should be ready for re-review

@bmtcril bmtcril mentioned this pull request Jan 18, 2024
@Ian2012 Ian2012 merged commit b043fa1 into main Jan 18, 2024
6 checks passed
@Ian2012 Ian2012 deleted the bmtcril/upgrade_db_load branch January 18, 2024 17:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants