Skip to content

Remove usages of deprecated VirtualTable.values field #23608

Description

@vbarua

The Substrait VirtualTable.values is deprecated and set to be removed in substrait-io/substrait#1131.

The DataFusion substrait producer still uses it when producing plans in

ReadType::VirtualTable(VirtualTable {
// Use deprecated 'values' field instead of 'expressions' because the consumer's
// nested expression support (RexType::Nested) is not yet implemented.
// The 'values' field uses literal::Struct which the consumer can properly
// deserialize with field name preservation.
#[expect(deprecated)]
values: vec![LiteralStruct { fields }],
expressions: vec![],
})

and
read_type: Some(ReadType::VirtualTable(VirtualTable {
values,
expressions,
})),

Usages of VirtualTable.values should be replaced with VirtualTable.expressions.

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Fields

No fields configured for issues without a type.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions