Fix like matching with newlines and no wildcard

presto-docs/src/main/sphinx/functions/comparison.rst

@@ -158,7 +158,7 @@ LIKE
 ----
 The LIKE operator is used to match a specified character pattern in a string. Patterns can contain
 regular characters as well as wildcards. Wildcard characters can be escaped using the single character
-specified for the ESCAPE parameter. Matching is case sensitive.
+specified for the ESCAPE parameter. Matching is case sensitive, and the pattern must match the whole string.
 
 Syntax:
 
@@ -199,6 +199,9 @@ Examples::
     WHERE name LIKE '%#%%' ESCAPE '#'
     --returns 'a%c' and  '%cd'
 
+   SELECT 'ab' || chr(10) || 'c' LIKE 'ab'  --chr(10) is a newline character
+    --returns 'false'
+
 
 Row comparison: IN
 ------------------

presto-main/src/main/java/com/facebook/presto/type/LikeFunctions.java

@@ -212,7 +212,10 @@ private static Regex likePattern(String patternString, char escapeChar, boolean
         regex.append('$');
 
         byte[] bytes = regex.toString().getBytes(UTF_8);
-        return new Regex(bytes, 0, bytes.length, Option.MULTILINE, NonStrictUTF8Encoding.INSTANCE, SYNTAX);
+        // Option.MULTILINE specifies that wildcard characters (. and *) should match newlines
+        // Option.SINGLELINE specifies that anchors (^ and $) should match the beginning and end of
+        // input rather than the beginning and end of the line
+        return new Regex(bytes, 0, bytes.length, Option.MULTILINE | Option.SINGLELINE, NonStrictUTF8Encoding.INSTANCE, SYNTAX);
     }
 
     @SuppressWarnings("NumericCastThatLosesPrecision")

presto-main/src/test/java/com/facebook/presto/sql/TestLikeFunctions.java

@@ -96,12 +96,32 @@ public void testLikeNewlineBeforeMatch()
         assertTrue(likeVarchar(utf8Slice("foo\nbar"), regex));
     }
 
+    @Test
+    public void testLikeNewlineNoWildcard()
+    {
+        Regex regex = likePattern(utf8Slice("foo\nbar"));
+        assertTrue(likeVarchar(utf8Slice("foo\nbar"), regex));
+    }
+
+    @Test
+    public void testLikeNoMatchAfterNewline()
+    {
+        Regex regex = likePattern(utf8Slice("foo"));
+        assertFalse(likeVarchar(utf8Slice("foo\nbar"), regex));
+    }
+
     @Test
     public void testLikeNewlineInMatch()
     {
         Regex regex = likePattern(utf8Slice("f%b%"));
         assertTrue(likeVarchar(utf8Slice("foo\nbar"), regex));
     }
+    @Test
+    public void testLikeNewlineInSingleWildcardMatch()
+    {
+        Regex regex = likePattern(utf8Slice("foo_bar"));
+        assertTrue(likeVarchar(utf8Slice("foo\nbar"), regex));
+    }
 
     @Test(timeOut = 1000)
     public void testLikeUtf8Pattern()