We are using non-string partition columns in production as well. Thanks, Aniket
On Sat, May 26, 2012 at 1:20 AM, Philip Tromans <philip.j.trom...@gmail.com>wrote: > We're using non-string partition columns in production. I think non string > partition columns are a good thing to have - it allows you to do all sorts > of date range calculations etc. AFAIK, MySQL's partition columns can be of > any type. > > Phil. > On May 26, 2012 7:55 AM, "Namit Jain" <nj...@fb.com> wrote: > > > Should I go ahead and file a jira to disallow non-string partition > columns > > ? > > Or, someone depends on that functionality. > > > > > > On 5/25/12 10:01 AM, "Namit Jain" <nj...@fb.com> wrote: > > > > >Yes, but the meta-question is: > > > > > >Is anyone dependent on non-string partition columns ? Should we drop the > > >support for non-string > > >partition columns ? > > > > > > > > >Thanks, > > >-namit > > > > > >On 5/24/12 11:21 PM, "Ashutosh Chauhan" <hashut...@apache.org> wrote: > > > > > >>Some discussion for this has happened on > > >>https://issues.apache.org/jira/browse/HIVE-2702 Is the underlying > > problem > > >>same as the one which I described on that jira ? > > >> > > >>Thanks, > > >>Ashutosh > > >> > > >>On Thu, May 24, 2012 at 10:59 PM, Namit Jain <nj...@fb.com> wrote: > > >> > > >>> Should we disallow non-string partition columns completely ? > > >>> Does anyone depend on that ? > > >>> > > >>> > > >>> On 5/24/12 6:49 PM, "Namit Jain" <nj...@fb.com> wrote: > > >>> > > >>> > > > >>> >http://svn.apache.org/viewvc?view=revision&revision=1308427 > > >>> > > > >>> >The patch above broke drop partitions if the partition happens to be > > >>> >non-string. > > >>> >This is due to a JDO issue with non-string columns. > > >>> > > > >>> >Is anyone using non-string partition columns ? > > >>> >Should be force the partition columns to be only of type string ? > > >>> >The documentation probably does not specify anything clearly. > > >>> > > > >>> >If someone is dependent on non-string partition column, we need to > > >>>revert > > >>> >this patch, or make a > > >>> >special case for string partition columns. > > >>> > > > >>> >Thanks, > > >>> >-namit > > >>> > > > >>> > > > >>> > > >>> > > > > > > > > -- "...:::Aniket:::... Quetzalco@tl"
