> On April 7, 2015, 4:45 a.m., Swarnim Kulkarni wrote: > > ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g, line 635 > > <https://reviews.apache.org/r/32809/diff/1/?file=914560#file914560line635> > > > > This and the following line can be simplified as > > > > return input.indexOf(c); > > pengcheng xiong wrote: > The purpose of the function is to test whether a string contains a char. > The actual index is only used to check if it is there, the detailed position > information is not needed. That is to say, a boolean return value is enough > for my purpose.
Yup. I think your logic is correct. Just having a return input.indexOf(c); serves the same purpose but is simpler. :) - Swarnim ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32809/#review79121 ----------------------------------------------------------- On April 7, 2015, 6:18 p.m., pengcheng xiong wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32809/ > ----------------------------------------------------------- > > (Updated April 7, 2015, 6:18 p.m.) > > > Review request for hive, Ashutosh Chauhan and John Pullokkaran. > > > Repository: hive-git > > > Description > ------- > > Since we don't allow users to query column names with dot in the middle such > as emp.no, don't allow users to create tables with such columns that cannot > be queried. Fix the documentation to reflect this fix. > > > Diffs > ----- > > ql/src/java/org/apache/hadoop/hive/ql/parse/HiveParser.g 2e583da > > ql/src/test/org/apache/hadoop/hive/ql/parse/TestUnpermittedCharsInColumnNameCreateTableNegative.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/32809/diff/ > > > Testing > ------- > > > Thanks, > > pengcheng xiong > >
