AryanBagade commented on issue #19039:
URL: https://github.com/apache/datafusion/issues/19039#issuecomment-3606609220
Hey @alamb
I have created a PR and implemented a builder API, following ur second
approach (required parameters in the constructor)
I went with:
```rust
CreateExternalTable::builder(name, location, file_type,
schema)
.with_options(options)
.build()
```
This enforces the 4 core required fields at compile-time, while optional
fields can be set via .with_*() methods (following the pattern I saw in
DataFusionErrorBuilder).
--
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]
