Andrew Lamb comments

Results 1880 comments of


                                            Andrew Lamb

Further refine the Top K sort operator

> Coming back to this, I guess if we can implement another option without implementing spilling: force compaction once we hit the limit. This probably slows down some queries if...

Further refine the Top K sort operator

Implementing this "reduce memory usage when under pressure" might be a more interesting general approach to improve DataFusion's performance under memory pressure (e.g. maybe we can trigger other operators to...

Converted internal representation of LogicalPlanBuilder from LogicalPlan to Arc<LogicalPlan> #10485

Thanks @iiiancampbell ! I triggered the CI to start

Converted internal representation of LogicalPlanBuilder from LogicalPlan to Arc<LogicalPlan> #10485

> Could you point me in the correct direction by any chance? I think you will need to check out the code and make the relevant changes to get the...

Converted internal representation of LogicalPlanBuilder from LogicalPlan to Arc<LogicalPlan> #10485

Converting to draft as we work to fix CI

`analysis.rs` bounds check panic

I think the analysis code is fairly unused at this point. Getting it more ship shape will be great.

`analysis.rs` bounds check panic

(Thank you so much for filing these issues)

`analysis.rs` bounds check panic

I agree that changing to return an internal error sounds like a much better plan

Using Union's input schema when recompute schema

Marking as draft as I think this PR is no longer waiting on feedback.

fix: change map root type to comply with https://github.com/apache/parquet-format/blob/master/LogicalTypes.md#maps to use key_value instead of entries.

> @alamb @tustvold are there any flags in Parquet-rs to make it transparently write map types with compliant naming? Not that I know of. I think it will typically write...