gstvg commented on PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2661146353
Many thanks @jayzhan211 @samuelcolvin @Omega359 @alamb :pray:
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use
alamb commented on PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2659540056
This PR caused a CI failure on main (due to a logical conflict). Small PR to
fix:
- https://github.com/apache/datafusion/pull/14667
--
This is an automated message from the Apach
alamb merged PR #12116:
URL: https://github.com/apache/datafusion/pull/12116
--
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...@datafusi
alamb commented on PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2659498648
Thanks again @gstvg @jayzhan211 and @Omega359
--
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 #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2656193648
It is exciting to get better union support as well
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL abo
Omega359 commented on PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2655362249
@alamb should be, yes.
--
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 c
alamb commented on PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2655006559
What is the status of this PR? Is is ready for a look?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1947939409
##
datafusion/expr/src/udf.rs:
##
@@ -951,4 +952,10 @@ The following regular expression functions are
supported:"#,
label: "Other Functions",
desc
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1947937272
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,260 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
Omega359 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1946472535
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,260 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
Omega359 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1946471266
##
datafusion/expr/src/udf.rs:
##
@@ -951,4 +952,10 @@ The following regular expression functions are
supported:"#,
label: "Other Functions",
d
gstvg commented on PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2601470429
Thanks @jayzhan211 🙏, this is ready for review
cc @tobixdev
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and
gstvg commented on PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2601112958
Hi @jayzhan211, can you reopen this, or should I open a new PR? Sorry for
the delay. Thanks!
--
This is an automated message from the Apache Git Service.
To respond to the message, p
github-actions[bot] closed pull request #12116: Add union_extract scalar
function
URL: https://github.com/apache/datafusion/pull/12116
--
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 co
github-actions[bot] commented on PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2469435968
Thank you for your contribution. Unfortunately, this pull request is stale
because it has been open 60 days with no activity. Please remove the stale
label or comment or
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1757226342
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1757226342
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1757226342
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1757194244
##
datafusion/sqllogictest/test_files/union_datatype.slt:
##
@@ -0,0 +1,255 @@
+# Licensed to the Apache Software Foundation (ASF) under one
Review Comment:
Look
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1757193466
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1757193028
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1755879897
##
datafusion/functions/benches/union_extract.rs:
##
@@ -0,0 +1,1121 @@
+#[macro_use]
+extern crate criterion;
+
+use crate::criterion::Criterion;
+use arrow::{
+
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1755878566
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1755877101
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1755876225
##
datafusion/sqllogictest/src/test_context.rs:
##
@@ -361,3 +369,893 @@ fn create_example_udf() -> ScalarUDF {
adder,
)
}
+
+fn register_union_tables
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1755875475
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
gstvg commented on PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#issuecomment-2344991858
Thanks @samuelcolvin and @jayzhan211 for your reviews. I moved part of this
PR to https://github.com/apache/arrow-rs/pull/6387. I made all your requests
that were applicable there, an
jayzhan211 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1731981156
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor l
jayzhan211 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1731980126
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor l
jayzhan211 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1731979504
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor l
jayzhan211 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1731979504
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor l
jayzhan211 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1731979153
##
datafusion/functions/benches/union_extract.rs:
##
@@ -0,0 +1,1121 @@
+#[macro_use]
+extern crate criterion;
+
+use crate::criterion::Criterion;
+use arrow::{
jayzhan211 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1731976320
##
datafusion/sqllogictest/src/test_context.rs:
##
@@ -361,3 +369,893 @@ fn create_example_udf() -> ScalarUDF {
adder,
)
}
+
+fn register_union_t
jayzhan211 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1731975403
##
datafusion/sqllogictest/test_files/union_datatype.slt:
##
@@ -0,0 +1,255 @@
+# Licensed to the Apache Software Foundation (ASF) under one
Review Comment:
jayzhan211 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1731975403
##
datafusion/sqllogictest/test_files/union_datatype.slt:
##
@@ -0,0 +1,255 @@
+# Licensed to the Apache Software Foundation (ASF) under one
Review Comment:
jayzhan211 commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1731970698
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor l
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1729378926
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
samuelcolvin commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1729195750
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1727611010
##
datafusion/functions/benches/union_extract.rs:
##
@@ -0,0 +1,1121 @@
+#[macro_use]
+extern crate criterion;
+
+use crate::criterion::Criterion;
+use arrow::{
+
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1727614121
##
datafusion/sqllogictest/src/test_context.rs:
##
@@ -361,3 +369,893 @@ fn create_example_udf() -> ScalarUDF {
adder,
)
}
+
+fn register_union_tables
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1727611010
##
datafusion/functions/benches/union_extract.rs:
##
@@ -0,0 +1,1121 @@
+#[macro_use]
+extern crate criterion;
+
+use crate::criterion::Criterion;
+use arrow::{
+
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1727611010
##
datafusion/functions/benches/union_extract.rs:
##
@@ -0,0 +1,1121 @@
+#[macro_use]
+extern crate criterion;
+
+use crate::criterion::Criterion;
+use arrow::{
+
gstvg commented on code in PR #12116:
URL: https://github.com/apache/datafusion/pull/12116#discussion_r1727606208
##
datafusion/functions/src/core/union_extract.rs:
##
@@ -0,0 +1,722 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor licens
43 matches
Mail list logo