Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/10174 )
Change subject: IMPALA-3282: [DOCS] Adds regexp_escape built-in function ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/10174/1/docs/topics/impala_string_functions.xml File docs/topics/impala_string_functions.xml: http://gerrit.cloudera.org:8080/#/c/10174/1/docs/topics/impala_string_functions.xml@705 PS1, Line 705: .\+*?[^]$(){}=!<>|:- Need to put this in CDATA. <> characters confuses XML converter. http://gerrit.cloudera.org:8080/#/c/10174/1/docs/topics/impala_string_functions.xml@705 PS1, Line 705: RE2's special characters: .\+*?[^]$(){}=!<>|:- so that the special characters are interpreted literally rather than as special characters. http://gerrit.cloudera.org:8080/#/c/10174/1/docs/topics/impala_string_functions.xml@718 PS1, Line 718: [localhost:21000] > select regexp_escape('Hello.world'); : +------------------------------------------------------+ : | regexp_escape('Hello.world') | : +------------------------------------------------------+ : | Hello\.world | : +------------------------------------------------------+ : Returned 1 row(s) in 0.12s</codeblock> : <p> : This example shows escaping all the special characters in RE2. : </p> : <codeblock>[localhost:21000] > select regexp_escape('a.b\\c+d*e?f[g]h$i(j)k{l}m=n!o<p>q|r:s-t'); : +------------------------------------------------------------+ : | regexp_escape('a.b\\c+d*e?f[g]h$i(j)k{l}m=n!o<p>q|r:s-t') | : +------------------------------------------------------------+ : | a\.b\\c\+d\*e\?f\[g\]h\$i\(j\)k\{l\}m\=n\!o\<p\>q\|r\:s\-t | : +------------------------------------------------------------+ : Returned 1 row(s) in 0.12s Put this in CDATA, as well. -- To view, visit http://gerrit.cloudera.org:8080/10174 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ied8e757c1b3012dd170b05da190d1598004d12cf Gerrit-Change-Number: 10174 Gerrit-PatchSet: 1 Gerrit-Owner: Kim Jin Chul <jinc...@gmail.com> Gerrit-Reviewer: Alex Rodoni <arod...@cloudera.com> Gerrit-Comment-Date: Tue, 24 Apr 2018 17:44:59 +0000 Gerrit-HasComments: Yes