[ https://issues.apache.org/jira/browse/KYLIN-3135?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16359768#comment-16359768 ]
Kaige Liu commented on KYLIN-3135: ----------------------------------- As commented above, I will revert the change in [https://github.com/apache/kylin/commit/7d5fb855064e2b81cd3b154cdeeafec4e64f63c9] Please review the patch [~roger.shi] [~lidong_sjtu], Thanks. > Fix regular expression bug in SQL comments > ------------------------------------------ > > Key: KYLIN-3135 > URL: https://issues.apache.org/jira/browse/KYLIN-3135 > Project: Kylin > Issue Type: Bug > Reporter: hahayuan > Assignee: hahayuan > Priority: Major > Fix For: v2.3.0 > > Attachments: 0001-KYLIN-3135.patch, 0002-KYLIN-3135-fix-regEx.patch, > multi_line_comments.PNG, one_line_comments.PNG > > > Hi,all. > 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 again. > And I was confused that the results says it was "No Support Sql",but it can > query success without comments. > For example, > {code:java} > /* > select count(*) from kylin_sales; > */ > select * from kylin_sales; > {code} > So I view the code and find the commentPatterns of /\**/ was > {code:java} > /\\*[^\\*/]* > {code} > ,clearly it was wrong. > 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, > {code:java} > /\\*[\\s\\S]*?\\*/ > {code} > if you think it's necessary to change the old code,please review and replace > it. > Thank for you time. -- This message was sent by Atlassian JIRA (v7.6.3#76005)