avantgardnerio commented on code in PR #501:
URL: https://github.com/apache/arrow-ballista/pull/501#discussion_r1018630373
##########
ballista/core/src/config.rs:
##########
@@ -87,6 +87,17 @@ impl BallistaConfigBuilder {
Self { settings }
}
+ pub fn load_env(&self) -> Self {
+ let mut settings = self.settings.clone();
+ if let Ok(it) = env::var("DATAFUSION_CATALOG_LOCATION") {
+ settings.insert("datafusion.catalog.location".to_string(), it);
Review Comment:
> Flight SQL page in the user guide should be updated
I can do that, but I think it should work across the board - even if you
connect from a regular ballista client. I'll test that.
> configs.md page should be updated with these DataFusion settings
Thanks, I was unaware. I'll go do that.
--
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]
