From 25768669f0d634197e7ca22cecc2d0bcb26c53e4 Mon Sep 17 00:00:00 2001
From: Michael Xu <xzhseh@gmail.com>
Date: Fri, 15 Sep 2023 16:11:38 -0400
Subject: [PATCH] handle possible panic case when building the regex

---
 e2e_test/batch/basic/func.slt.part | 3 +++
 src/expr/src/vector_op/regexp.rs   | 4 +++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/e2e_test/batch/basic/func.slt.part b/e2e_test/batch/basic/func.slt.part
index ec60f6512406d..501ab464b8acd 100644
--- a/e2e_test/batch/basic/func.slt.part
+++ b/e2e_test/batch/basic/func.slt.part
@@ -594,6 +594,9 @@ select regexp_replace('foobarXaz', '(?<!X)a', 'X');
 ----
 foobXrXaz
 
+query error Parsing error at position 2: Unknown group flag: (?1
+select regexp_replace('foobarXaz', '(?1:123)', 'X');
+
 query T
 select regexp_count('ABCABCAXYaxy', 'A.');
 ----
diff --git a/src/expr/src/vector_op/regexp.rs b/src/expr/src/vector_op/regexp.rs
index 85b4be213fd6e..0962d0bf16c4f 100644
--- a/src/expr/src/vector_op/regexp.rs
+++ b/src/expr/src/vector_op/regexp.rs
@@ -40,7 +40,9 @@ impl RegexpContext {
         };
 
         Ok(Self {
-            regex: RegexBuilder::new(&origin).build().unwrap(),
+            regex: RegexBuilder::new(&origin)
+                .build()
+                .map_err(|e| ExprError::Parse(e.to_string().into()))?,
             global: options.global,
             replacement: make_replacement(replacement),
         })