I cant recall all the details, but it was in there because of fix for a specific bug which we hit. We should leave it in there, unless its making some other changes really hard.
Ashutosh On Thu, Feb 9, 2012 at 02:55, Julien Le Dem <[email protected]> wrote: > In HCatBasedStorer (line 428) there is a check to verify field aliases are > all lower case. > What's the reason for this ? It seems unnecessary, can we remove it ? > thank you > Julien > > if(alias.matches(".*[A-Z]+.*")) { > throw new FrontendException("Column names should all be in lowercase. > Invalid name found: "+alias, PigHCatUtil.PIG_EXCEPTION_CODE); > } > > >
