2010YOUY01 merged PR #15594:
URL: https://github.com/apache/datafusion/pull/15594
--
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-unsubscr...@dat
alamb commented on PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#issuecomment-2813926979
🚀 so exciting -- thank you
--
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
alamb commented on PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#issuecomment-2810565181
I merged up from main to rerun CI to be sure
--
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 g
alamb commented on PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#issuecomment-2810564535
We made `branch-47` so let's get this one in!
--
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
alamb commented on PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#issuecomment-2807139683
Thanks -- I recommend we merge this after the 47.0.0 release is created
(likely tomorrow)
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
2010YOUY01 commented on code in PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#discussion_r2041433804
##
datafusion/core/src/execution/context/mod.rs:
##
@@ -1036,13 +1040,73 @@ impl SessionContext {
variable, value, ..
} = stmt;
-
kumarlokesh commented on code in PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#discussion_r2040707194
##
datafusion/core/tests/sql/runtime_config.rs:
##
@@ -0,0 +1,73 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lice
2010YOUY01 commented on code in PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#discussion_r2032386011
##
datafusion/core/tests/sql/runtime_config.rs:
##
@@ -0,0 +1,73 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licen
kumarlokesh commented on PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#issuecomment-2784505817
> Hello @kumarlokesh. Thank you for working on this. I have 2
questions/concerns. Let's discuss on them a bit to get a future-proof design
>
> 1. There are also `runtime_en
berkaysynnada commented on PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#issuecomment-2786609340
Thank you @kumarlokesh for addressing my comments. I don't have further
suggestions or concern other than @2010YOUY01 shared
--
This is an automated message from the Apache G
berkaysynnada commented on code in PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#discussion_r2033296553
##
datafusion/core/src/execution/context/mod.rs:
##
@@ -1036,13 +1040,82 @@ impl SessionContext {
variable, value, ..
} = stmt;
-
berkaysynnada commented on PR #15594:
URL: https://github.com/apache/datafusion/pull/15594#issuecomment-2781398502
Hello @kumarlokesh. Thank you for working on this. I have 2
questions/concerns. Let's discuss on them a bit to get a future-proof design
1) There are also `runtime_env: A
12 matches
Mail list logo
