Skip to content

The limit info lost in the AggregateExec when ser/deser the physical plan  #10630

@liukun4515

Description

@liukun4515

Describe the bug

In the definition of AggregateExec, we have the limit argument to optimize the agg operation.

pub struct AggregateExec {
     .....
    /// Set if the output of this aggregation is truncated by a upstream sort/limit clause
    limit: Option<usize>,
    .....
}

But it is not added in the .proto file, when we use the proto to ser/deser the agg physical node, the limit will lost.

To Reproduce

No response

Expected behavior

No response

Additional context

add limit option in the proto file, and add implementation for the limit arg in the to_proto and from_proto

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions