Skip to content

Combine utilities in SpillManager #16907

@2010YOUY01

Description

@2010YOUY01

Is your feature request related to a problem or challenge?

follow-up to: #15700

This is a simple clean-up idea, see the original discussion in:
#15700 (comment)

spill_record_batch_by_size_and_return_max_batch_memory() and spill_record_batch_by_size() can be combined to re-use the code (perhaps only keep the first one)

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions