alamb commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-3039007195
🚀
--
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
blaginin commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-3033917307
@nirnayroy thank you so much for your work, i think that's a very useful new
function 🎉 welcome to the project!!!
--
This is an automated message from the Apache Git Service.
To r
blaginin merged PR #15928:
URL: https://github.com/apache/datafusion/pull/15928
--
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...@dataf
Omega359 commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-3033041600
LGTM :) @blaginin
--
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 comme
Omega359 commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-3032478116
Run extended tests
--
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 comme
Omega359 commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-3029308432
Run extended tests
--
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 comme
Omega359 commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-302984
Clippy failures related to rand update (I think
https://github.com/apache/datafusion/pull/16062)
--
This is an automated message from the Apache Git Service.
To respond to the mes
Omega359 commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-3029112924
Run extended tests
--
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 comme
alamb commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-3029119907
> Run extended tests
It works!
--
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 th
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2180625721
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,826 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
Omega359 commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-3025423773
I'll pull this branch later this week and run the tests but in general this
PR is looking pretty good! I left a few comments/suggestions for a few things I
found from a quick review
Omega359 commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2178434012
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,826 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lice
Omega359 commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2178426472
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,826 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lice
Omega359 commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2178423784
##
datafusion/functions/src/regex/mod.rs:
##
@@ -89,7 +122,44 @@ pub fn functions() -> Vec> {
vec![
regexp_count(),
regexp_match(),
+
Omega359 commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2178421432
##
datafusion/functions/src/regex/mod.rs:
##
@@ -60,6 +65,34 @@ pub mod expr_fn {
super::regexp_match().call(args)
}
+/// Returns index of reg
nirnayroy commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-3015598680
Hi @blaginin , thanks for the help and suggestions for improvement.
I have addressed the requested changes. Please have another look.
> Tests are failing. If that helps, yo
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2173375931
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,804 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2173329256
##
datafusion/functions/benches/regx.rs:
##
@@ -127,6 +128,46 @@ fn criterion_benchmark(c: &mut Criterion) {
})
});
+c.bench_function("regexp
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2159984385
##
datafusion/functions/src/regex/regexpcount.rs:
##
@@ -29,10 +30,10 @@ use datafusion_expr::{
use datafusion_macros::user_doc;
use itertools::izip;
use rege
blaginin commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2155683469
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,804 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lice
blaginin commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2155683469
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,804 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lice
blaginin commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2155673833
##
datafusion/functions/src/regex/regexpcount.rs:
##
@@ -29,10 +30,10 @@ use datafusion_expr::{
use datafusion_macros::user_doc;
use itertools::izip;
use regex
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2154895463
##
datafusion/functions/src/regex/regexpcount.rs:
##
@@ -550,7 +550,7 @@ where
}
}
-fn compile_and_cache_regex<'strings, 'cache>(
+pub fn compile_and_cac
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2154383754
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,804 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
blaginin commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2153128215
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,804 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lice
Copilot commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2153125391
##
datafusion/functions/benches/regx.rs:
##
@@ -127,6 +128,46 @@ fn criterion_benchmark(c: &mut Criterion) {
})
});
+c.bench_function("regexp_i
blaginin commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2153122396
##
datafusion/functions/src/regex/regexpcount.rs:
##
@@ -550,7 +550,7 @@ where
}
}
-fn compile_and_cache_regex<'strings, 'cache>(
+pub fn compile_and_cach
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2141872552
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,979 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2141878573
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,979 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
nirnayroy commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-2965393471
Hi @blaginin, thanks for the review and regret the delay in reply. I think I
have rectified a majority of the concerns raised. Please have a look again.
--
This is an automated
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2141879046
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,979 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2141870238
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,979 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2141882989
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,979 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2141880357
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,979 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2141877745
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,979 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
nirnayroy commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2141871203
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,979 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lic
alamb commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-2898907668
@Omega359 I wonder if you might have time to review this PR?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use t
blaginin commented on code in PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#discussion_r2101076229
##
datafusion/functions/src/regex/regexpinstr.rs:
##
@@ -0,0 +1,979 @@
+// Licensed to the Apache Software Foundation (ASF) under one
+// or more contributor lice
Omega359 commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-2899035444
Of course @alamb, not sure how I missed this one. It may be a day or two
though
--
This is an automated message from the Apache Git Service.
To respond to the message, please log
nirnayroy commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-2893026630
fixed formatting error in workflow
--
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 t
nirnayroy commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-289230
fixed the cippy errors showing up in the workflow
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
nirnayroy commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-2890882937
> Thank you for this PR @nirnayroy
>
> Can you please resolve the CI error:
https://github.com/apache/datafusion/actions/runs/14820525339/job/41754009017?pr=15928
>
>
alamb commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-2860276419
Can you please resolve the CI error:
https://github.com/apache/datafusion/actions/runs/14820525339/job/41754009017?pr=15928
> # If you encounter an error, run './dev/update_funct
nirnayroy commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-2849046029
> Thank you. I'm wondering what's the reference system for this function's
behavior (like postgres or others)
The reference system for this function's behaviour is
[post
2010YOUY01 commented on PR #15928:
URL: https://github.com/apache/datafusion/pull/15928#issuecomment-2848931103
Thank you. I'm wondering what's the reference system for this function's
behavior (like postgres or others)
--
This is an automated message from the Apache Git Service.
To respo
45 matches
Mail list logo