Mailing-List: contact dev-help@hive.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hive.apache.org
Received-SPF: pass (nike.apache.org: domain of brock@cloudera.com designates
 209.85.216.42 as permitted sender)
MIME-Version: 1.0
In-Reply-To: 
 <CAG0AUg2a8jmGd4iyQ-hbRXfuBFCfH_Zy=Kc80JBX-3uhut7GOQ@mail.gmail.com>
References: 
 <CAFukC=6S6NfwsP7nefURUL5bbTd-TgAdGZ7x37FSvOpWix-VKg@mail.gmail.com>
 <CAG0AUg2NZvJ2fYsBQVN7_61y0SmwqUu_4JND+xpBz30CNUM8QQ@mail.gmail.com>
 <CAFukC=4CuCLqYA_0pCaJZT1GqH8WDDBDUS_EXKcmi-DLUm=ENw@mail.gmail.com>
 <CAG0AUg2a8jmGd4iyQ-hbRXfuBFCfH_Zy=Kc80JBX-3uhut7GOQ@mail.gmail.com>
From: Brock Noland <brock@cloudera.com>
Date: Wed, 16 Oct 2013 12:40:33 -0500
Message-ID: 
 <CAFukC=69b=tODRcHk2yabcwAQ3iTrWOvbOdgG80zsY4BFYoVAQ@mail.gmail.com>
Subject: Re: Why aren't both HCatalog tests being updated?
To: "dev@hive.apache.org" <dev@hive.apache.org>
Content-Type: text/plain; charset=ISO-8859-1

It sounds like we are just forgetting. Please consider this a friendly
reminder to update both tests if you are fixing some environmental or
bug in the tests.

On Wed, Oct 16, 2013 at 12:36 PM, Eugene Koifman
<ekoifman@hortonworks.com> wrote:
> I don't have a good answer for this
>
>
> On Wed, Oct 16, 2013 at 9:58 AM, Brock Noland <brock@cloudera.com> wrote:
>
>> Agreed, but notice that the changes I reference are fixes for tests.
>> Why wouldn't they be replicated?
>>
>> Example:
>>
>>
>> https://github.com/apache/hive/blob/trunk/hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatPartitionPublish.java#L160
>>
>> https://github.com/apache/hive/blob/trunk/hcatalog/core/src/test/java/org/apache/hcatalog/mapreduce/TestHCatPartitionPublish.java#L153
>>
>> Example:
>>
>>
>> https://github.com/apache/hive/blob/trunk/hcatalog/webhcat/java-client/src/test/java/org/apache/hive/hcatalog/api/TestHCatClient.java#L122
>>
>> https://github.com/apache/hive/blob/trunk/hcatalog/webhcat/java-client/src/test/java/org/apache/hcatalog/api/TestHCatClient.java#L125
>>
>> On Wed, Oct 16, 2013 at 11:55 AM, Eugene Koifman
>> <ekoifman@hortonworks.com> wrote:
>> > As a general rule, everything in org.apache.hcatalog is deprecated and
>> > frozen at 0.11 version.  All new features/bug fixes should be made in
>> > org.apache.hive.hcatalog.
>> >
>> >
>> > On Wed, Oct 16, 2013 at 7:52 AM, Brock Noland <brock@cloudera.com>
>> wrote:
>> >
>> >> Hi,
>> >>
>> >> I've noticed a number of cases where one HCatalog test is modified but
>> >> the deprecated "pair" is not. Are we just forgetting or is this on
>> >> purpose?
>> >>
>> >> Example:
>> >>
>> >>
>> >>
>> https://github.com/apache/hive/blob/trunk/hcatalog/core/src/test/java/org/apache/hcatalog/mapreduce/TestHCatPartitionPublish.java#L153
>> >>
>> >>
>> https://github.com/apache/hive/blob/trunk/hcatalog/core/src/test/java/org/apache/hcatalog/mapreduce/TestHCatPartitionPublish.java#L153
>> >>
>> >> Example:
>> >>
>> >>
>> >>
>> https://github.com/apache/hive/blob/trunk/hcatalog/webhcat/java-client/src/test/java/org/apache/hive/hcatalog/api/TestHCatClient.java#L122
>> >>
>> >>
>> https://github.com/apache/hive/blob/trunk/hcatalog/webhcat/java-client/src/test/java/org/apache/hcatalog/api/TestHCatClient.java#L125
>> >>
>> >> Brock
>> >>
>> >
>> > --
>> > 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
>> immediately
>> > and delete it from your system. Thank You.
>>
>>
>>
>> --
>> Apache MRUnit - Unit testing MapReduce - http://mrunit.apache.org
>>
>
> --
> 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 immediately
> and delete it from your system. Thank You.


-- 
Apache MRUnit - Unit testing MapReduce - http://mrunit.apache.org