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

[kkhatua]

Github user kkhatua commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1084#discussion_r160487621
  
    --- 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) {
    +"use strict";
    +
    +exports.snippetText = "snippet info\n\
    +   select * from INFORMATION_SCHEMA.${1:<tableName>};\n\
    +snippet sysmem\n\
    --- End diff --
    
    Not sure what you mean.
    All available snippet keywords (e.g. 'info') are in lower-case, but I don't 
think that is affecting the suggested autocomplete options. The template, 
however, is projected verbatim to the definition. So, INFORMATION_SCHEMA will 
remain in upper case. 
    
![image](https://user-images.githubusercontent.com/4335237/34736399-a2c01e78-f527-11e7-84aa-fcdfcefc6a5c.png)



---