Correct str_detect() translation (#384)

Fixes #369
r-dbi · May 5, 2020 · 35a80ad · 35a80ad
1 parent a258fd6
commit 35a80ad
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/NEWS.md b/NEWS.md
@@ -1,5 +1,8 @@
 # bigrquery (development version)
 
+* `str_detect()` now correctly translated to `REGEXP_CONTAINS`  
+  (@jimmyg3g, #369).
+
 * Error messages inlude hints for common problems (@deflaux, #353).
 
 * `bq_perform_upload()` works once again (#361). It seems like the generated

diff --git a/R/dplyr.R b/R/dplyr.R
@@ -174,7 +174,7 @@ sql_translate_env.BigQueryConnection <- function(x) {
       paste0 = sql_prefix("CONCAT"),
 
       # stringr equivalents
-      str_detect =  sql_prefix("REGEXP_MATCH", 2),
+      str_detect =  sql_prefix("REGEXP_CONTAINS", 2),
       str_extract = sql_prefix("REGEXP_EXTRACT", 2),
       str_replace = sql_prefix("REGEXP_REPLACE", 3),