Re: [PR] Add regexp_extract func [datafusion]

2025-04-02 Thread via GitHub
SKY-ALIN closed pull request #14282: Add regexp_extract func URL: https://github.com/apache/datafusion/pull/14282 -- 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] Add regexp_extract func [datafusion]

2025-04-02 Thread via GitHub
Omega359 commented on PR #14282: URL: https://github.com/apache/datafusion/pull/14282#issuecomment-2773275754 I spent some time today looking at this PR. Here are my thoughts: - This udf closely mirrors the [duckdb](https://duckdb.org/docs/stable/sql/functions/regular_expressions.html

Re: [PR] Add regexp_extract func [datafusion]

2025-04-02 Thread via GitHub
Omega359 commented on PR #14282: URL: https://github.com/apache/datafusion/pull/14282#issuecomment-2772580217 I'll take a look at this later today @SKY-ALIN -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

Re: [PR] Add regexp_extract func [datafusion]

2025-03-25 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r2012956967 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,322 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-03-21 Thread via GitHub
Omega359 commented on PR #14282: URL: https://github.com/apache/datafusion/pull/14282#issuecomment-2743165457 @SKY-ALIN - are you able to look into the above review comments? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Add regexp_extract func [datafusion]

2025-03-05 Thread via GitHub
rluvaton commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1981449667 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,322 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-02-02 Thread via GitHub
Omega359 commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1938507496 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,322 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-02-01 Thread via GitHub
rluvaton commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1938303236 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,322 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-02-01 Thread via GitHub
rluvaton commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1938301064 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-31 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1937604422 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-31 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1937745027 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-31 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1937744642 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-31 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1937744088 ## docs/source/user-guide/sql/scalar_functions.md: ## @@ -1919,6 +1920,31 @@ SELECT regexp_replace('aBc', '(b|d)', 'Ab\\1a', 'i'); Additional examples can be f

Re: [PR] Add regexp_extract func [datafusion]

2025-01-31 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1937604422 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-28 Thread via GitHub
rluvaton commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1932896635 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-28 Thread via GitHub
rluvaton commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1932901931 ## docs/source/user-guide/sql/scalar_functions.md: ## @@ -1919,6 +1920,31 @@ SELECT regexp_replace('aBc', '(b|d)', 'Ab\\1a', 'i'); Additional examples can be f

Re: [PR] Add regexp_extract func [datafusion]

2025-01-28 Thread via GitHub
rluvaton commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1932904240 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-28 Thread via GitHub
rluvaton commented on PR #14282: URL: https://github.com/apache/datafusion/pull/14282#issuecomment-2620107324 FYI Spark regex is not the same as Rust regex and can have different results -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [PR] Add regexp_extract func [datafusion]

2025-01-27 Thread via GitHub
alamb commented on PR #14282: URL: https://github.com/apache/datafusion/pull/14282#issuecomment-2616949171 FYI @Omega359 noted this is quite similar to - https://github.com/apache/datafusion/pull/14323 -- This is an automated message from the Apache Git Service. To respond to the mess

Re: [PR] Add regexp_extract func [datafusion]

2025-01-26 Thread via GitHub
Omega359 commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1929809676 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-25 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1929597131 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-25 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1929596877 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-25 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1929596970 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-25 Thread via GitHub
SKY-ALIN commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1929596852 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-25 Thread via GitHub
Omega359 commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1929541426 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-25 Thread via GitHub
Omega359 commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1929541028 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-25 Thread via GitHub
Omega359 commented on PR #14282: URL: https://github.com/apache/datafusion/pull/14282#issuecomment-2613964912 Thanks for your contribution! I've left some comments for your review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH

Re: [PR] Add regexp_extract func [datafusion]

2025-01-25 Thread via GitHub
Omega359 commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1929541738 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li

Re: [PR] Add regexp_extract func [datafusion]

2025-01-25 Thread via GitHub
Omega359 commented on code in PR #14282: URL: https://github.com/apache/datafusion/pull/14282#discussion_r1929540823 ## datafusion/functions/src/regex/regexpextract.rs: ## @@ -0,0 +1,289 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor li