Tom Ebergen

@Mytherin I should have addressed most of the comments. I had to refactor the estimator because one of the failing tests revealed a weakness in the estimation logic, this issue...

@Mytherin Attempted to change the estimated_cardinality to type `idx_t` but was getting regressions in a number of JOB queries. If I change line 284 in `cardinality_estimator.cpp` from `return numerator /...

@david-cortes can you verify if this is still an issue? Running this on 0.8.1 doesn't produce any errors

> Looks much more readable now with the refactor, thanks! > > Are the changes to the `ColumnLifetimeAnalyzer` on purpose, or were you doing that on another branch? > >...

Running CI locally to see if the Java failure was spurious or not https://github.com/Tmonster/duckdb/pull/131

Hi qzbqzb, Can you provide details on the following? - Number of cores available to Duckdb - The number of rows in files data_[12-16].parquet I have found that duckdb estimates...

@qzbqzb Can you let me know if the new version of Duckdb (v0.10.0) fixes your issue? Thanks!

closing since @qzbqzb reported the issue as fixed

Hi James, Does this issue persist on duckdb v0.10.0? if so, could you also paste the query plans here. From what I can tell in the codebase, the two queries...

@jamesryanbell please let me know if you have any updates. Would like to solve this for you and others who might be experiencing the issue.