erenavsarogullari opened a new pull request, #20375:
URL: https://github.com/apache/datafusion/pull/20375
## Which issue does this PR close?
- Closes 20373.
## Rationale for this change
Minor refactoring on error message by exposing required config name for the
end user. This is follow-up PR to both PR: #20226 and #20372 by runtime config:
`datafusion.runtime.max_temp_directory_size`.
## What changes are included in this PR?
**Current:**
```
The used disk space during the spilling process has exceeded the allowable
limit of {}. Try increasing the `max_temp_directory_size` in the disk manager
configuration.
```
**New:**
```
The used disk space during the spilling process has exceeded the allowable
limit of {}. \
Please try increasing the config:
`datafusion.runtime.max_temp_directory_size`.",
```
## Are these changes tested?
Yes, legacy UT case has been updated by covering expected config name.
## Are there any user-facing changes?
Yes, error message has been updated which is exposed to end-users.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
