-
Notifications
You must be signed in to change notification settings - Fork 4.6k
🎉 New Destination: DuckDB #17494
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
Merged
Merged
🎉 New Destination: DuckDB #17494
Changes from 38 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
0a6f0fc
init commit: step 4 `check` works. 5 `write` WIP
sspaeti 9c336ea
update: write is working now
sspaeti 8c9af94
Try/error installing DuckDB inside docker
sspaeti 4becb2a
fixing installing duckdb
sspaeti f3c5c3e
adding local folder
sspaeti 0d3a20a
change path to magic folder /local/
sspaeti f9b245e
remove if/else case: not needed anymore
sspaeti b799d7b
adding docs
sspaeti a1a61f1
adding tests
sspaeti 653685e
normalization: first batch (integration tests are not working)
sspaeti 6a7774c
fixing integration and unit tests
sspaeti 2809bfb
updated builds.md and some minor
sspaeti 8934da7
add destinationDefinitionId and fix minor in docs
sspaeti ffc1c8f
fix normalizations errors
sspaeti c993ed3
fixing some errors
sspaeti 786df8a
add info box for shared files
sspaeti 9bcc41c
fixing dbt_profile instead of `destination_path` it must be `path` (s…
sspaeti 22d9a54
fix null handling
sspaeti 8a375e9
Merge master
sspaeti 06fd245
update svg to square
sspaeti 4d3d87f
adding gradle
sspaeti 6c39be2
Merge branch 'master' into duckdb-destination
sspaeti ce972bb
adding new normalization definitions
sspaeti 1b3fbb8
formatting CONNECTORS_BASE build
sspaeti 9a7cbd6
merge master into branch
sspaeti d2e7ac7
adding properties of destination_path and schema to new destinaton_sp…
sspaeti 76cfe5d
formatter
sspaeti 8224d8f
fix `properties` in wrong indention
sspaeti faa8697
merge master into branch
sspaeti 311f589
use integration test as not secrets are saved
sspaeti 975dfb2
commiting a first patch from review
sspaeti e819731
part two from review
sspaeti 4502beb
formatter and added copyright back
sspaeti 1b6e8be
revert message3
sspaeti c36efa2
result after debugging. Reverting close connection and max identifier…
sspaeti 2b0ec8f
merge master
sspaeti 97a7462
gradle format
sspaeti 71fe767
adding AirbyteStateMessageTest
sspaeti 7b0bfef
Merge master
sspaeti 23b02b7
adding duckdb after merge conflict: changed from 'cloudfront.net/test…
sspaeti File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Loading
Sorry, something went wrong. Reload?
Sorry, we cannot display this file.
Sorry, this file is invalid so it cannot be displayed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
63 changes: 63 additions & 0 deletions
63
airbyte-integrations/bases/base-normalization/dbt-project-template-duckdb/dbt_project.yml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,63 @@ | ||
# This file is necessary to install dbt-utils with dbt deps | ||
# the content will be overwritten by the transform function | ||
|
||
# Name your package! Package names should contain only lowercase characters | ||
# and underscores. A good package name should reflect your organization's | ||
# name or the intended use of these models | ||
name: "airbyte_utils" | ||
version: "1.0" | ||
config-version: 2 | ||
|
||
# This setting configures which "profile" dbt uses for this project. Profiles contain | ||
# database connection information, and should be configured in the ~/.dbt/profiles.yml file | ||
profile: "normalize" | ||
|
||
# These configurations specify where dbt should look for different types of files. | ||
# The `model-paths` config, for example, states that source models can be found | ||
# in the "models/" directory. You probably won't need to change these! | ||
model-paths: ["models"] | ||
docs-paths: ["docs"] | ||
analysis-paths: ["analysis"] | ||
test-paths: ["tests"] | ||
seed-paths: ["data"] | ||
macro-paths: ["macros"] | ||
|
||
target-path: "../build" # directory which will store compiled SQL files | ||
log-path: "../logs" # directory which will store DBT logs | ||
packages-install-path: "/dbt" # directory which will store external DBT dependencies | ||
|
||
clean-targets: # directories to be removed by `dbt clean` | ||
- "build" | ||
- "dbt_modules" | ||
|
||
quoting: | ||
database: true | ||
# Temporarily disabling the behavior of the ExtendedNameTransformer on table/schema names, see (issue #1785) | ||
# all schemas should be unquoted | ||
schema: false | ||
identifier: true | ||
|
||
# You can define configurations for models in the `model-paths` directory here. | ||
# Using these configurations, you can enable or disable models, change how they | ||
# are materialized, and more! | ||
models: | ||
airbyte_utils: | ||
+materialized: table | ||
generated: | ||
airbyte_ctes: | ||
+tags: airbyte_internal_cte | ||
+materialized: ephemeral | ||
airbyte_incremental: | ||
+tags: incremental_tables | ||
+materialized: incremental | ||
+on_schema_change: sync_all_columns | ||
airbyte_tables: | ||
+tags: normalized_tables | ||
+materialized: table | ||
airbyte_views: | ||
+tags: airbyte_internal_views | ||
+materialized: view | ||
|
||
dispatch: | ||
- macro_namespace: dbt_utils | ||
search_order: ["airbyte_utils", "dbt_utils"] |
5 changes: 5 additions & 0 deletions
5
airbyte-integrations/bases/base-normalization/dbt-project-template-duckdb/packages.yml
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
# add dependencies. these will get pulled during the `dbt deps` process. | ||
|
||
packages: | ||
- git: "https://github.com/fishtown-analytics/dbt-utils.git" | ||
revision: 0.8.2 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
after we publish normalization, we should bump this version to point at the new version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you publish normalization, or how do I do that?