Re: [PR] Implementation for regex_instr [datafusion]

2025-07-05 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-03 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-03 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-03 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-03 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-02 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-02 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-02 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-02 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-02 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-01 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-01 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-01 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-01 Thread via GitHub
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(), +

Re: [PR] Implementation for regex_instr [datafusion]

2025-07-01 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-28 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-28 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-28 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-21 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-18 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-18 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-18 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-18 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-18 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-17 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-17 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-17 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-12 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-12 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-12 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-12 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-12 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-12 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-12 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-12 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-06-12 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-05-21 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-05-21 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-05-21 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-05-19 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-05-19 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-05-19 Thread via GitHub
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 > >

Re: [PR] Implementation for regex_instr [datafusion]

2025-05-07 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-05-03 Thread via GitHub
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

Re: [PR] Implementation for regex_instr [datafusion]

2025-05-03 Thread via GitHub
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