You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@drill.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/03/08 01:13:40 UTC

[jira] [Commented] (DRILL-4486) Expression serializer incorrectly serializes escaped characters

    [ https://issues.apache.org/jira/browse/DRILL-4486?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15184071#comment-15184071 ] 

ASF GitHub Bot commented on DRILL-4486:
---------------------------------------

GitHub user StevenMPhillips opened a pull request:

    https://github.com/apache/drill/pull/412

    DRILL-4486: Fix expression serialization escaping

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/StevenMPhillips/drill parser

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/drill/pull/412.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #412
    
----
commit 26c6f97e08698f91298186d9c545ea1fe3102552
Author: Steven Phillips <st...@dremio.com>
Date:   2016-03-05T04:29:26Z

    DRILL-4486: Fix expression serialization escaping

----


> Expression serializer incorrectly serializes escaped characters
> ---------------------------------------------------------------
>
>                 Key: DRILL-4486
>                 URL: https://issues.apache.org/jira/browse/DRILL-4486
>             Project: Apache Drill
>          Issue Type: Bug
>            Reporter: Steven Phillips
>            Assignee: Steven Phillips
>
> the drill expression parser requires backslashes to be escaped. But the ExpressionStringBuilder is not properly escaping them. This causes problems, especially in the case of regex expressions run with parallel execution.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)