kylin-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "hahayuan (JIRA)" <>
Subject [jira] [Created] (KYLIN-3135) regular expression of multi line comment pattern was wrong
Date Wed, 27 Dec 2017 16:07:00 GMT
hahayuan created KYLIN-3135:

             Summary: regular expression of multi line comment pattern was wrong
                 Key: KYLIN-3135
             Project: Kylin
          Issue Type: Bug
            Reporter: hahayuan

Recently,I was testing query function of kylin,
sometimes I just comment with /**/ instead of delete the sql,cause I need to query and compare
And I was confused that the results says it was "No Support Sql",but it can query success
without comments.
For example,
select count(*) from kylin_sales;
select * from kylin_sales;
So I view the code and find the commentPatterns of /**/ was "/\\*[^\\*/]*",clearly it was
The regular expression of [abc] means any character in abc,such as a or b.
So the [^\\*/] means that * or / can't appear,
But under this circumstances the */ need to be as a string not separated character.
the */ can't appear not * or / can't appear.
I rewrite the regular expression,"/\\*[\\s\\S]*?\\*/",if you think it's necessary,you can
review and replace it.

This message was sent by Atlassian JIRA

View raw message