forked from duckdb/duckdb
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Dependency manager powered export #82
Draft
Tishj
wants to merge
267
commits into
logical_dependency
Choose a base branch
from
dependency_manager_powered_export
base: logical_dependency
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Conversation
This file contains 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
…recreated by GetInfo in the CatalogEntry
…pendency where possible, especially CheckpointManager is interesting here because for attached databases the catalog is not available through Catalog::GetEntry
…es are not supported anyways
Due to random failure connected to cert, possibly see hendrikmuhs/ccache-action#184
…agation Fix missing opener propagation
Fix duckdb#11246: Use SetConsoleCP function to set input to UTF8 when reading
CLI: Add support for ".edit" or "\e"
Fix VS2022 Preview ClangCl build
Remove an unnecessary line from bind_insert.cpp
[CI] Skip ccache for R.yml
Improve binding of CTEs
…inder Move BindCreateIndex from Catalog to Binder
[Substrait-ADBC] Fix for substrait plan execution via ADBC
Removing abort() from RE2 again because Google refuses to use exceptions
Defer allocation in read_json
…asis, as such they cannot exceed the STANDARD_VECTOR_SIZE and we do not need to use 8-byte identifiers for them but can use a uint16_t instead
[ODBC] Add escape character to ParseStringFilter to support Power Query ('table_name' is escaped to 'table\_name')
Bump to post-portfile change for duckdb_azure
Reduce memory usage of DELETE operations
Use `optional_idx` in more places
…index-on-binder Revert "Move BindCreateIndex from Catalog to Binder"
[Arrow] Throw on invalid STRUCT type
[Dev] Do not use CatalogEntry references inside Dependency objects.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Continuation on #81
This PR will:
schema.sql
order duckdb/duckdb#8619We introduce LogicalDependency as a precursor to PhysicalDependency, this merely contains the information needed to look up the catalog entry, so it's safe to serialize/deserialize
Inside
SingleFileCheckpointWriter::CreateCheckpoint()
there was a duplicate of GetNaiveExportOrder, this now uses theDependencyManager::GetExportOrder
methodViews now create a dependency on the table they're referencing (as long as they are in the same catalog)
This changes some existing behavior, but is in line with postgres behavior
1: Indexes are not included in this, this requires some additional work that needs to be done outside of this PR