Skip to content

Commit

Permalink
update ComplexFilterPushdown()
Browse files Browse the repository at this point in the history
  • Loading branch information
stephaniewang526 committed Aug 21, 2024
1 parent e0af7f6 commit 8de8d0d
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 6 deletions.
2 changes: 1 addition & 1 deletion duckdb
Submodule duckdb updated 3558 files
8 changes: 5 additions & 3 deletions src/functions/delta_scan.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -438,13 +438,15 @@ void DeltaSnapshot::InitializeFiles() {
initialized = true;
}

unique_ptr<MultiFileList> DeltaSnapshot::ComplexFilterPushdown(ClientContext &context, const MultiFileReaderOptions &options, LogicalGet &get,
vector<unique_ptr<Expression>> &filters) {
unique_ptr<MultiFileList> DeltaSnapshot::ComplexFilterPushdown(ClientContext &context,
const MultiFileReaderOptions &options,
MultiFilePushdownInfo &info,

Check failure on line 443 in src/functions/delta_scan.cpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / MacOS (osx_amd64, x86_64, x64-osx)

unknown type name 'MultiFilePushdownInfo'

Check failure on line 443 in src/functions/delta_scan.cpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / MacOS (osx_arm64, arm64, arm64-osx)

unknown type name 'MultiFilePushdownInfo'

Check failure on line 443 in src/functions/delta_scan.cpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / Linux (linux_amd64, ubuntu:18.04, x64-linux)

'MultiFilePushdownInfo' has not been declared

Check failure on line 443 in src/functions/delta_scan.cpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / Linux (linux_amd64, ubuntu:18.04, x64-linux)

'MultiFilePushdownInfo' has not been declared
vector<unique_ptr<Expression>> &filters) {
FilterCombiner combiner(context);
for (const auto &filter : filters) {
combiner.AddFilter(filter->Copy());
}
auto filterstmp = combiner.GenerateTableScanFilters(get.column_ids);
auto filterstmp = combiner.GenerateTableScanFilters(info.column_ids);

Check failure on line 449 in src/functions/delta_scan.cpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / Linux (linux_amd64, ubuntu:18.04, x64-linux)

request for member 'column_ids' in 'info', which is of non-class type 'int'

Check failure on line 449 in src/functions/delta_scan.cpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / Linux (linux_amd64, ubuntu:18.04, x64-linux)

request for member 'column_ids' in 'info', which is of non-class type 'int'

// TODO: can/should we figure out if this filtered anything?
auto filtered_list = make_uniq<DeltaSnapshot>(context, paths[0]);
Expand Down
5 changes: 3 additions & 2 deletions src/include/functions/delta_scan.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -43,8 +43,9 @@ struct DeltaSnapshot : public MultiFileList {
public:
void Bind(vector<LogicalType> &return_types, vector<string> &names);
unique_ptr<MultiFileList> ComplexFilterPushdown(ClientContext &context,

Check failure on line 45 in src/include/functions/delta_scan.hpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / Linux (linux_amd64, ubuntu:18.04, x64-linux)

'duckdb::unique_ptr<duckdb::MultiFileList, std::default_delete<duckdb::MultiFileList>, true> duckdb::DeltaSnapshot::ComplexFilterPushdown(duckdb::ClientContext&, const duckdb::MultiFileReaderOptions&, int&, duckdb::vector<duckdb::unique_ptr<duckdb::Expression> >&)' marked 'override', but does not override

Check failure on line 45 in src/include/functions/delta_scan.hpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / Linux (linux_amd64, ubuntu:18.04, x64-linux)

'duckdb::unique_ptr<duckdb::MultiFileList, std::default_delete<duckdb::MultiFileList>, true> duckdb::DeltaSnapshot::ComplexFilterPushdown(duckdb::ClientContext&, const duckdb::MultiFileReaderOptions&, int&, duckdb::vector<duckdb::unique_ptr<duckdb::Expression> >&)' marked 'override', but does not override
const MultiFileReaderOptions &options, LogicalGet &get,
vector<unique_ptr<Expression>> &filters) override;
const MultiFileReaderOptions &options,
MultiFilePushdownInfo &info,

Check failure on line 47 in src/include/functions/delta_scan.hpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / MacOS (osx_amd64, x86_64, x64-osx)

unknown type name 'MultiFilePushdownInfo'

Check failure on line 47 in src/include/functions/delta_scan.hpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / MacOS (osx_arm64, arm64, arm64-osx)

unknown type name 'MultiFilePushdownInfo'

Check failure on line 47 in src/include/functions/delta_scan.hpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / Linux (linux_amd64, ubuntu:18.04, x64-linux)

'MultiFilePushdownInfo' has not been declared

Check failure on line 47 in src/include/functions/delta_scan.hpp

View workflow job for this annotation

GitHub Actions / Build extension binaries / Linux (linux_amd64, ubuntu:18.04, x64-linux)

'MultiFilePushdownInfo' has not been declared
vector<unique_ptr<Expression>> &filters) override;
vector<string> GetAllFiles() override;
FileExpandResult GetExpandResult() override;
idx_t GetTotalFileCount() override;
Expand Down

0 comments on commit 8de8d0d

Please sign in to comment.