Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Hive >> mail # dev >> HCatalog Refactoring - no checkins into hive/hcatalog part of the src tree


Copy link to this message
-
Re: HCatalog Refactoring - no checkins into hive/hcatalog part of the src tree
Alright, HIVE-5236 is up.

I tried using the eclipse auto-formatter using the hive prefs - that
was a world of pain in and of itself. So, with a bunch of perl regex
replacements, I've generated a new patch. If others can review this,
and are on board that we want to do this, it's available.

On Thu, Sep 5, 2013 at 7:46 PM, Brock Noland <[EMAIL PROTECTED]> wrote:
> +1 the difference between the two projects is painful.
>
> On Sep 5, 2013 9:32 PM, "Thejas Nair" <[EMAIL PROTECTED]> wrote:
>>
>> Its also a great time to fix indentation of hcatalog and make it
>> comply with rest of hive . Any volunteers ? I hope the IDE can do it
>> with a few clicks if the project is setup right.
>>
>>
>> On Thu, Sep 5, 2013 at 7:21 PM, Sushanth Sowmyan <[EMAIL PROTECTED]>
>> wrote:
>> > Okay, HIVE-5233 is also resolved now, and hcatalog/ and subdirs are
>> > now completely unfrozen. Thanks for the patches, Eugene!
>> >
>> > On Thu, Sep 5, 2013 at 6:15 PM, Sushanth Sowmyan <[EMAIL PROTECTED]>
>> > wrote:
>> >> As an update, it is now 6pm PDT on September 5th, and we're now done
>> >> with the changes in HIVE-4895 and HIVE-4896.
>> >>
>> >> However, there is one more issue we found along the way, and that's
>> >> addressed by HIVE-5233 which I still have to review before committing.
>> >>
>> >> hcatalog/ itself can be considered unfrozen, but we still need to
>> >> consider hcatalog/storage-handlers/hbase/ frozen till 11:59pm PDT on
>> >> September 5th.
>> >>
>> >> Since this directory only deals with code that is now considered
>> >> deprecated, I hope it won't affect anyone (except Viraj from
>> >> HIVE-4331) but that patch is not in a patch-available state.
>> >>
>> >>
>> >>
>> >> On Tue, Sep 3, 2013 at 6:50 PM, Sushanth Sowmyan <[EMAIL PROTECTED]>
>> >> wrote:
>> >>> Sounds good, go for it. Any hcatalog patch-available patches that we
>> >>> do not get in by 8AM PDT will have to be canceled and regenerated.
>> >>> I'll do that tomorrow.
>> >>>
>> >>> On Tue, Sep 3, 2013 at 6:34 PM, Eugene Koifman
>> >>> <[EMAIL PROTECTED]> wrote:
>> >>>> Yes, that is correct.
>> >>>>
>> >>>>
>> >>>> On Tue, Sep 3, 2013 at 6:28 PM, Thejas Nair <[EMAIL PROTECTED]>
>> >>>> wrote:
>> >>>>
>> >>>>> Eugene,
>> >>>>> I assume you want the hold off on checkins to hive/hcatalog dir to
>> >>>>> start from tomorrow AM pacific time (Sept 4) (say 8 am PDT ?) and
>> >>>>> you
>> >>>>> expect you changes to be done by Sept 5 around 6pm Pacific time. Is
>> >>>>> that correct ?
>> >>>>>
>> >>>>> Thanks,
>> >>>>> Thejas
>> >>>>>
>> >>>>>
>> >>>>>
>> >>>>> On Tue, Sep 3, 2013 at 6:00 PM, Eugene Koifman
>> >>>>> <[EMAIL PROTECTED]>
>> >>>>> wrote:
>> >>>>> > Hi,
>> >>>>> >
>> >>>>> > the work on 2 items (described in detail earlier on this mail
>> >>>>> > list) below
>> >>>>> > will start tomorrow. Please avoid checking anything into
>> >>>>> > hive/hcatalog. I
>> >>>>> > expect this to be complete by EOD Thursday.
>> >>>>> >
>> >>>>> >
>> >>>>> > 1. HIVE-4895 - change package name from org.apache.hcatalog to
>> >>>>> > org.apache.hive.hcatalog
>> >>>>> >
>> >>>>> > 2. HIVE-4896 - create binary (API) backwards compatibility layer
>> >>>>> > for hcat
>> >>>>> > users upgrading from 0.11 to 0.12
>> >>>>> >
>> >>>>> >
>> >>>>> > Thank you,
>> >>>>> >
>> >>>>> > Eugene
>> >>>>> >
>> >>>>> > --
>> >>>>> > CONFIDENTIALITY NOTICE
>> >>>>> > NOTICE: This message is intended for the use of the individual or
>> >>>>> > entity
>> >>>>> to
>> >>>>> > which it is addressed and may contain information that is
>> >>>>> > confidential,
>> >>>>> > privileged and exempt from disclosure under applicable law. If the
>> >>>>> > reader
>> >>>>> > of this message is not the intended recipient, you are hereby
>> >>>>> > notified
>> >>>>> that
>> >>>>> > any printing, copying, dissemination, distribution, disclosure or
>> >>>>> > forwarding of this communication is strictly prohibited. If you
>> >>>>> > have
>> >>>>> > received this communication in error, please contact the sender