milenkovicm merged PR #1230:
URL: https://github.com/apache/datafusion-ballista/pull/1230
--
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: github-unsubsc
milenkovicm commented on PR #1230:
URL:
https://github.com/apache/datafusion-ballista/pull/1230#issuecomment-2815461468
Thanks @joaoferrao
I agree with you. As a follow up I will try to enable s3 in default context,
once i merge few other PRs. Will look into improving doc as well.
milenkovicm commented on PR #1230:
URL:
https://github.com/apache/datafusion-ballista/pull/1230#issuecomment-2799983961
@mmooyyii & @joaoferrao would you mind have a look at this PR
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to G
milenkovicm commented on code in PR #1230:
URL:
https://github.com/apache/datafusion-ballista/pull/1230#discussion_r2033058510
##
ballista/core/src/object_store.rs:
##
@@ -86,10 +86,10 @@ pub fn custom_runtime_env_with_s3_support(
///
/// It will configure [SessionState] with
joaoferrao commented on code in PR #1230:
URL:
https://github.com/apache/datafusion-ballista/pull/1230#discussion_r2033050934
##
ballista/core/src/object_store.rs:
##
@@ -86,10 +86,10 @@ pub fn custom_runtime_env_with_s3_support(
///
/// It will configure [SessionState] with
joaoferrao commented on code in PR #1230:
URL:
https://github.com/apache/datafusion-ballista/pull/1230#discussion_r2033050934
##
ballista/core/src/object_store.rs:
##
@@ -86,10 +86,10 @@ pub fn custom_runtime_env_with_s3_support(
///
/// It will configure [SessionState] with