-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
"Wrong number of children" #9895
Comments
If you can post a
|
@mustafasrepo This is the childrends function, I'll get grab a plan in a bit pub struct DeltaScan {
/// The URL of the ObjectStore root
pub table_uri: String,
/// Column that contains an index that maps to the original metadata Add
pub config: DeltaScanConfig,
/// The parquet scan to wrap
pub parquet_scan: Arc<dyn ExecutionPlan>,
/// The schema of the table to be used when evaluating expressions
pub logical_schema: Arc<ArrowSchema>,
}
fn children(&self) -> Vec<Arc<dyn ExecutionPlan>> {
vec![self.parquet_scan.clone()]
}
fn with_new_children(
self: Arc<Self>,
children: Vec<Arc<dyn ExecutionPlan>>,
) -> DataFusionResult<Arc<dyn ExecutionPlan>> {
ExecutionPlan::with_new_children(self.parquet_scan.clone(), children)
} |
@ion-elgreco try changing the children function to return an empty |
I think, changing implementation below
to something like
might solve the problem (Just brainstorming, I didn't try it). I am not sure, what |
@mustafasrepo thanks for the tips! It's resolved now |
Describe the bug
A couple folks including me tried bumping Datafusion to v36 in Delta-RS but to no avail since we all are running in this
Wrong number of children
error. Which at that time I read is something in the optimizer?Here are failing actions: https://github.com/delta-io/delta-rs/actions/runs/8339625658/job/22821968902#step:4:1
https://github.com/delta-io/delta-rs/actions/runs/8472960661/job/23243814572#step:4:1
I'm curious if you guys would know the issue, perhaps it's something that we missed
To Reproduce
Not sure, but our tests will failed in this branch: delta-io/delta-rs#2214
Expected behavior
No response
Additional context
No response
The text was updated successfully, but these errors were encountered: