Skip to content

Commit b4a8b5a

Browse files
authored
Add PhysicalExpr::is_volatile_node to upgrade guide (#17443)
1 parent 50e073c commit b4a8b5a

File tree

1 file changed

+18
-0
lines changed

1 file changed

+18
-0
lines changed

docs/source/library-user-guide/upgrading.md

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -285,6 +285,24 @@ If you have custom implementations of `FileOpener` or work directly with `FileOp
285285

286286
[#17397]: https://github.com/apache/datafusion/pull/17397
287287

288+
### Added `PhysicalExpr::is_volatile_node`
289+
290+
We added a method to `PhysicalExpr` to mark a `PhysicalExpr` as volatile:
291+
292+
```rust,ignore
293+
impl PhysicalExpr for MyRandomExpr {
294+
fn is_volatile_node(&self) -> bool {
295+
true
296+
}
297+
}
298+
```
299+
300+
We've shipped this with a default value of `false` to minimize breakage but we highly recommend that implementers of `PhysicalExpr` opt into a behavior, even if it is returning `false`.
301+
302+
You can see more discussion and example implementations in [#17351].
303+
304+
[#17351]: https://github.com/apache/datafusion/pull/17351
305+
288306
## DataFusion `49.0.0`
289307

290308
### `MSRV` updated to 1.85.1

0 commit comments

Comments
 (0)