[GitHub] drill pull request #1084: DRILL-5868: Support SQL syntax highlighting of que...

[arina-ielchiieva]

Github user arina-ielchiieva commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1084#discussion_r160449936
  
    --- Diff: 
exec/java-exec/src/main/resources/rest/static/js/ace-code-editor/snippets/sql.js
 ---
    @@ -0,0 +1,46 @@
    +/**
    + * Drill SQL Syntax Snippets
    + */
    +
    +ace.define("ace/snippets/sql",["require","exports","module"], 
function(require, exports, module) {
    --- End diff --
    
    1. Could you please add information about snippets to Jira description. We 
might what to add it to Drill documentation later so user knows which snippets 
can be used. Let's say, it's hard to guess the snippers without looking into 
the code :)
    
    2. When using snippets, I saw the following warning.
    
    
![image](https://user-images.githubusercontent.com/15086720/34730558-6a34fde4-f568-11e7-81e9-3e6d62091189.png)
    Could you please check?
      


---