----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28173/#review62940 -----------------------------------------------------------
Ship it! Ship It! - Jason Altekruse On Nov. 21, 2014, 11:40 p.m., abdelhakim deneche wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28173/ > ----------------------------------------------------------- > > (Updated Nov. 21, 2014, 11:40 p.m.) > > > Review request for drill. > > > Bugs: DRILL-1545 > https://issues.apache.org/jira/browse/DRILL-1545 > > > Repository: drill-git > > > Description > ------- > > updated JSONFormatPlugin, adding an extensions array into JSONFormatConfig. I > used TextFormatPlugin as a reference. > Now the user can set the following format in Web UI: > "json": { > "type": "json", > "extensions": [ > "json", > "log" > ] > } > > The user can also write a separate config for json and log like this: > "json": { > "type": "json", > "extensions": [ > "json" > ] > }, > "log": { > "type": "json", > "extensions": [ > "log" > ] > } > > > Diffs > ----- > > > exec/java-exec/src/main/java/org/apache/drill/exec/store/easy/json/JSONFormatPlugin.java > e1165a2 > exec/java-exec/src/main/resources/bootstrap-storage-plugins.json 4a20bea > > Diff: https://reviews.apache.org/r/28173/diff/ > > > Testing > ------- > > created two copies of the same .json data file, one with .json extension and > another with .log extension. > > updated "json" format like this: > > "json": { > "type": "json", > "extensions": [ > "json", > "log" > ] > } > > once the change was done, I was able to query both files from sqlline. > > > Thanks, > > abdelhakim deneche > >
