----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/26117/#review55177 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/plan/ShowFunctionsDesc.java <https://reviews.apache.org/r/26117/#comment95565> I think it is bit cleaner to have this and other constructors call the new constructor. In this case it is not as important as the constructor is doing very little, but it is good to follow that patter to avoid code repetition. ql/src/test/queries/clientpositive/show_functions.q <https://reviews.apache.org/r/26117/#comment95566> can you also add a test that involves both | and * ? - Thejas Nair On Sept. 27, 2014, 9:01 p.m., peter liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/26117/ > ----------------------------------------------------------- > > (Updated Sept. 27, 2014, 9:01 p.m.) > > > Review request for hive and Thejas Nair. > > > Bugs: HIVE-8094 > https://issues.apache.org/jira/browse/HIVE-8094 > > > Repository: hive-git > > > Description > ------- > > It would be nice to add LIKE keyword support for SHOW FUNCTIONS as below, > and keep the patterns consistent to the way as SHOW DATABASES, SHOW TABLES. > > bq. SHOW FUNCTIONS LIKE 'foo*'; > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java b5e3ff0 > ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java f17b93d > ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java > eb20be3 > ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 37832be > ql/src/java/org/apache/hadoop/hive/ql/plan/ShowFunctionsDesc.java 5ea99f8 > ql/src/test/queries/clientpositive/show_functions.q c09f50a > ql/src/test/results/clientpositive/show_functions.q.out 775cc74 > > Diff: https://reviews.apache.org/r/26117/diff/ > > > Testing > ------- > > > Thanks, > > peter liu > >
