You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by Brock Noland <br...@cloudera.com> on 2012/11/16 00:16:33 UTC

[VOTE] Release Apache Flume version 1.3.0-rc3

This is the second release for Apache Flume as a top-level project,
version 1.3.0. We are voting on release candidate rc3.

*** Please cast your vote within the next 72 hours ***

The list of fixed issues:
https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3

The tarball (*.tar.gz), signature (*.asc), and checksums (*.md5, *.sha1)
for the source and binary artifacts can be found at:
http://people.apache.org/~brock/flume-1.3.0-rc3/

(Note: attempting to deploy the artifacts to Nexus appears to have
locked me out of Apache LDAP so you'll have to use the artifacts from
the above link or via the source tag below.)

The tag to be voted on:
https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3

The KEYS file can be found here:
https://dist.apache.org/repos/dist/release/flume/KEYS

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Roshan Naik <ro...@hortonworks.com>.
yup. i tried it on a different box myself.. it built cleanly. So ..
+1

-roshan


On Fri, Nov 16, 2012 at 4:33 PM, Brock Noland <br...@cloudera.com> wrote:

> I tried the again one two different systems (mac os lion and rhel 6.3)
> and was unable to reproduce so I am guessing the issues are
> environmental.
>
> On Fri, Nov 16, 2012 at 5:51 PM, Brock Noland <br...@cloudera.com> wrote:
> > Can you try again or see if there is some environmental issue? I
> > cannot reproduce locally and jenkins ran find on all of last nights
> > commits:
> >
> > https://builds.apache.org/job/flume-trunk/328/
> >
> > Brock
> >
> >
> > On Fri, Nov 16, 2012 at 5:46 PM, Roshan Naik <ro...@hortonworks.com>
> wrote:
> >> I see the below errors.  Based on following Latest commit...
> >> commit 656e049981df62316e0a817382087297209bf2f3
> >> Author: Brock Noland <br...@apache.org>
> >> Date:   Thu Nov 15 16:28:05 2012 -0600
> >>
> >>     FLUME-1716: Update CHANGELOG for rc3
> >>
> >>     (Brock Noland via Brock Noland)
> >>
> >>
> >>
> >> -------------------------------------------------------
> >>  T E S T S
> >> -------------------------------------------------------
> >> Running org.apache.flume.test.agent.TestFileChannel
> >> Tests run: 2, Failures: 0, *Errors: 2,* Skipped: 0, Time elapsed: 5.248
> sec
> >> <<< FAILURE!
> >> Running org.apache.flume.test.agent.TestRpcClient
> >> Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 9.854
> sec
> >> Running org.apache.flume.test.agent.TestRpcClientCommunicationFailure
> >> Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 8.413
> sec
> >>
> >> Results :
> >>
> >> Tests in error:
> >>   testInOut(org.apache.flume.test.agent.TestFileChannel): Cannot run
> >> program
> >>
> "/Users/roshan/Projects/flume-git-release/flume-ng-tests/target/test/flume7556416910922042256_stage_dir/apache-flume-1.3.0-bin/bin/flume-ng"
> >> (in directory
> >>
> "/Users/roshan/Projects/flume-git-release/flume-ng-tests/target/test/flume7556416910922042256_stage_dir/apache-flume-1.3.0-bin"):
> >> error=13, Permission denied
> >>   testInOut(org.apache.flume.test.agent.TestFileChannel): Process not
> found
> >>
> >>
> >>
> >> Tests run: 4, Failures: 0, Errors: 2, Skipped: 0
> >>
> >> [INFO]
> >> ------------------------------------------------------------------------
> >> [INFO] Reactor Summary:
> >> [INFO]
> >> [INFO] Apache Flume ...................................... SUCCESS
> [55.433s]
> >> [INFO] Flume NG SDK ...................................... SUCCESS
> [59.447s]
> >> [INFO] Flume NG Configuration ............................ SUCCESS
> [2.025s]
> >> [INFO] Flume NG Core ..................................... SUCCESS
> >> [2:49.995s]
> >> [INFO] Flume NG Sinks .................................... SUCCESS
> [1.032s]
> >> [INFO] Flume NG HDFS Sink ................................ SUCCESS
> [38.792s]
> >> [INFO] Flume NG IRC Sink ................................. SUCCESS
> [1.812s]
> >> [INFO] Flume NG HBase Sink ............................... SUCCESS
> >> [1:20.950s]
> >> [INFO] Flume NG ElasticSearch Sink ....................... SUCCESS
> [13.275s]
> >> [INFO] Flume Sources ..................................... SUCCESS
> [0.565s]
> >> [INFO] Flume Scribe Source ............................... SUCCESS
> [1.672s]
> >> [INFO] Flume NG Channels ................................. SUCCESS
> [1.101s]
> >> [INFO] Flume NG JDBC channel ............................. SUCCESS
> [42.526s]
> >> [INFO] Flume NG Node ..................................... SUCCESS
> [20.195s]
> >> [INFO] Flume NG file-based channel ....................... SUCCESS
> >> [4:11.444s]
> >> [INFO] Flume NG file backed Memory channel ............... SUCCESS
> [26.813s]
> >> [INFO] Flume legacy Sources .............................. SUCCESS
> [0.573s]
> >> [INFO] Flume legacy Avro source .......................... SUCCESS
> [2.035s]
> >> [INFO] Flume legacy Thrift Source ........................ SUCCESS
> [1.634s]
> >> [INFO] Flume NG Clients .................................. SUCCESS
> [0.538s]
> >> [INFO] Flume NG Log4j Appender ........................... SUCCESS
> [2.833s]
> >> [INFO] Flume NG distribution ............................. SUCCESS
> [22.601s]
> >> [INFO] Flume NG Integration Tests ........................ *FAILURE*
> >>  [27.408s]
> >> [INFO]
> >> ------------------------------------------------------------------------
> >
> >
> >
> > --
> > Apache MRUnit - Unit testing MapReduce -
> http://incubator.apache.org/mrunit/
>
>
>
> --
> Apache MRUnit - Unit testing MapReduce -
> http://incubator.apache.org/mrunit/
>

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Brock Noland <br...@cloudera.com>.
I tried the again one two different systems (mac os lion and rhel 6.3)
and was unable to reproduce so I am guessing the issues are
environmental.

On Fri, Nov 16, 2012 at 5:51 PM, Brock Noland <br...@cloudera.com> wrote:
> Can you try again or see if there is some environmental issue? I
> cannot reproduce locally and jenkins ran find on all of last nights
> commits:
>
> https://builds.apache.org/job/flume-trunk/328/
>
> Brock
>
>
> On Fri, Nov 16, 2012 at 5:46 PM, Roshan Naik <ro...@hortonworks.com> wrote:
>> I see the below errors.  Based on following Latest commit...
>> commit 656e049981df62316e0a817382087297209bf2f3
>> Author: Brock Noland <br...@apache.org>
>> Date:   Thu Nov 15 16:28:05 2012 -0600
>>
>>     FLUME-1716: Update CHANGELOG for rc3
>>
>>     (Brock Noland via Brock Noland)
>>
>>
>>
>> -------------------------------------------------------
>>  T E S T S
>> -------------------------------------------------------
>> Running org.apache.flume.test.agent.TestFileChannel
>> Tests run: 2, Failures: 0, *Errors: 2,* Skipped: 0, Time elapsed: 5.248 sec
>> <<< FAILURE!
>> Running org.apache.flume.test.agent.TestRpcClient
>> Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 9.854 sec
>> Running org.apache.flume.test.agent.TestRpcClientCommunicationFailure
>> Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 8.413 sec
>>
>> Results :
>>
>> Tests in error:
>>   testInOut(org.apache.flume.test.agent.TestFileChannel): Cannot run
>> program
>> "/Users/roshan/Projects/flume-git-release/flume-ng-tests/target/test/flume7556416910922042256_stage_dir/apache-flume-1.3.0-bin/bin/flume-ng"
>> (in directory
>> "/Users/roshan/Projects/flume-git-release/flume-ng-tests/target/test/flume7556416910922042256_stage_dir/apache-flume-1.3.0-bin"):
>> error=13, Permission denied
>>   testInOut(org.apache.flume.test.agent.TestFileChannel): Process not found
>>
>>
>>
>> Tests run: 4, Failures: 0, Errors: 2, Skipped: 0
>>
>> [INFO]
>> ------------------------------------------------------------------------
>> [INFO] Reactor Summary:
>> [INFO]
>> [INFO] Apache Flume ...................................... SUCCESS [55.433s]
>> [INFO] Flume NG SDK ...................................... SUCCESS [59.447s]
>> [INFO] Flume NG Configuration ............................ SUCCESS [2.025s]
>> [INFO] Flume NG Core ..................................... SUCCESS
>> [2:49.995s]
>> [INFO] Flume NG Sinks .................................... SUCCESS [1.032s]
>> [INFO] Flume NG HDFS Sink ................................ SUCCESS [38.792s]
>> [INFO] Flume NG IRC Sink ................................. SUCCESS [1.812s]
>> [INFO] Flume NG HBase Sink ............................... SUCCESS
>> [1:20.950s]
>> [INFO] Flume NG ElasticSearch Sink ....................... SUCCESS [13.275s]
>> [INFO] Flume Sources ..................................... SUCCESS [0.565s]
>> [INFO] Flume Scribe Source ............................... SUCCESS [1.672s]
>> [INFO] Flume NG Channels ................................. SUCCESS [1.101s]
>> [INFO] Flume NG JDBC channel ............................. SUCCESS [42.526s]
>> [INFO] Flume NG Node ..................................... SUCCESS [20.195s]
>> [INFO] Flume NG file-based channel ....................... SUCCESS
>> [4:11.444s]
>> [INFO] Flume NG file backed Memory channel ............... SUCCESS [26.813s]
>> [INFO] Flume legacy Sources .............................. SUCCESS [0.573s]
>> [INFO] Flume legacy Avro source .......................... SUCCESS [2.035s]
>> [INFO] Flume legacy Thrift Source ........................ SUCCESS [1.634s]
>> [INFO] Flume NG Clients .................................. SUCCESS [0.538s]
>> [INFO] Flume NG Log4j Appender ........................... SUCCESS [2.833s]
>> [INFO] Flume NG distribution ............................. SUCCESS [22.601s]
>> [INFO] Flume NG Integration Tests ........................ *FAILURE*
>>  [27.408s]
>> [INFO]
>> ------------------------------------------------------------------------
>
>
>
> --
> Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/



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

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Brock Noland <br...@cloudera.com>.
Can you try again or see if there is some environmental issue? I
cannot reproduce locally and jenkins ran find on all of last nights
commits:

https://builds.apache.org/job/flume-trunk/328/

Brock


On Fri, Nov 16, 2012 at 5:46 PM, Roshan Naik <ro...@hortonworks.com> wrote:
> I see the below errors.  Based on following Latest commit...
> commit 656e049981df62316e0a817382087297209bf2f3
> Author: Brock Noland <br...@apache.org>
> Date:   Thu Nov 15 16:28:05 2012 -0600
>
>     FLUME-1716: Update CHANGELOG for rc3
>
>     (Brock Noland via Brock Noland)
>
>
>
> -------------------------------------------------------
>  T E S T S
> -------------------------------------------------------
> Running org.apache.flume.test.agent.TestFileChannel
> Tests run: 2, Failures: 0, *Errors: 2,* Skipped: 0, Time elapsed: 5.248 sec
> <<< FAILURE!
> Running org.apache.flume.test.agent.TestRpcClient
> Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 9.854 sec
> Running org.apache.flume.test.agent.TestRpcClientCommunicationFailure
> Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 8.413 sec
>
> Results :
>
> Tests in error:
>   testInOut(org.apache.flume.test.agent.TestFileChannel): Cannot run
> program
> "/Users/roshan/Projects/flume-git-release/flume-ng-tests/target/test/flume7556416910922042256_stage_dir/apache-flume-1.3.0-bin/bin/flume-ng"
> (in directory
> "/Users/roshan/Projects/flume-git-release/flume-ng-tests/target/test/flume7556416910922042256_stage_dir/apache-flume-1.3.0-bin"):
> error=13, Permission denied
>   testInOut(org.apache.flume.test.agent.TestFileChannel): Process not found
>
>
>
> Tests run: 4, Failures: 0, Errors: 2, Skipped: 0
>
> [INFO]
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO]
> [INFO] Apache Flume ...................................... SUCCESS [55.433s]
> [INFO] Flume NG SDK ...................................... SUCCESS [59.447s]
> [INFO] Flume NG Configuration ............................ SUCCESS [2.025s]
> [INFO] Flume NG Core ..................................... SUCCESS
> [2:49.995s]
> [INFO] Flume NG Sinks .................................... SUCCESS [1.032s]
> [INFO] Flume NG HDFS Sink ................................ SUCCESS [38.792s]
> [INFO] Flume NG IRC Sink ................................. SUCCESS [1.812s]
> [INFO] Flume NG HBase Sink ............................... SUCCESS
> [1:20.950s]
> [INFO] Flume NG ElasticSearch Sink ....................... SUCCESS [13.275s]
> [INFO] Flume Sources ..................................... SUCCESS [0.565s]
> [INFO] Flume Scribe Source ............................... SUCCESS [1.672s]
> [INFO] Flume NG Channels ................................. SUCCESS [1.101s]
> [INFO] Flume NG JDBC channel ............................. SUCCESS [42.526s]
> [INFO] Flume NG Node ..................................... SUCCESS [20.195s]
> [INFO] Flume NG file-based channel ....................... SUCCESS
> [4:11.444s]
> [INFO] Flume NG file backed Memory channel ............... SUCCESS [26.813s]
> [INFO] Flume legacy Sources .............................. SUCCESS [0.573s]
> [INFO] Flume legacy Avro source .......................... SUCCESS [2.035s]
> [INFO] Flume legacy Thrift Source ........................ SUCCESS [1.634s]
> [INFO] Flume NG Clients .................................. SUCCESS [0.538s]
> [INFO] Flume NG Log4j Appender ........................... SUCCESS [2.833s]
> [INFO] Flume NG distribution ............................. SUCCESS [22.601s]
> [INFO] Flume NG Integration Tests ........................ *FAILURE*
>  [27.408s]
> [INFO]
> ------------------------------------------------------------------------



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

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Roshan Naik <ro...@hortonworks.com>.
I see the below errors.  Based on following Latest commit...
commit 656e049981df62316e0a817382087297209bf2f3
Author: Brock Noland <br...@apache.org>
Date:   Thu Nov 15 16:28:05 2012 -0600

    FLUME-1716: Update CHANGELOG for rc3

    (Brock Noland via Brock Noland)



-------------------------------------------------------
 T E S T S
-------------------------------------------------------
Running org.apache.flume.test.agent.TestFileChannel
Tests run: 2, Failures: 0, *Errors: 2,* Skipped: 0, Time elapsed: 5.248 sec
<<< FAILURE!
Running org.apache.flume.test.agent.TestRpcClient
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 9.854 sec
Running org.apache.flume.test.agent.TestRpcClientCommunicationFailure
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 8.413 sec

Results :

Tests in error:
  testInOut(org.apache.flume.test.agent.TestFileChannel): Cannot run
program
"/Users/roshan/Projects/flume-git-release/flume-ng-tests/target/test/flume7556416910922042256_stage_dir/apache-flume-1.3.0-bin/bin/flume-ng"
(in directory
"/Users/roshan/Projects/flume-git-release/flume-ng-tests/target/test/flume7556416910922042256_stage_dir/apache-flume-1.3.0-bin"):
error=13, Permission denied
  testInOut(org.apache.flume.test.agent.TestFileChannel): Process not found



Tests run: 4, Failures: 0, Errors: 2, Skipped: 0

[INFO]
------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Apache Flume ...................................... SUCCESS [55.433s]
[INFO] Flume NG SDK ...................................... SUCCESS [59.447s]
[INFO] Flume NG Configuration ............................ SUCCESS [2.025s]
[INFO] Flume NG Core ..................................... SUCCESS
[2:49.995s]
[INFO] Flume NG Sinks .................................... SUCCESS [1.032s]
[INFO] Flume NG HDFS Sink ................................ SUCCESS [38.792s]
[INFO] Flume NG IRC Sink ................................. SUCCESS [1.812s]
[INFO] Flume NG HBase Sink ............................... SUCCESS
[1:20.950s]
[INFO] Flume NG ElasticSearch Sink ....................... SUCCESS [13.275s]
[INFO] Flume Sources ..................................... SUCCESS [0.565s]
[INFO] Flume Scribe Source ............................... SUCCESS [1.672s]
[INFO] Flume NG Channels ................................. SUCCESS [1.101s]
[INFO] Flume NG JDBC channel ............................. SUCCESS [42.526s]
[INFO] Flume NG Node ..................................... SUCCESS [20.195s]
[INFO] Flume NG file-based channel ....................... SUCCESS
[4:11.444s]
[INFO] Flume NG file backed Memory channel ............... SUCCESS [26.813s]
[INFO] Flume legacy Sources .............................. SUCCESS [0.573s]
[INFO] Flume legacy Avro source .......................... SUCCESS [2.035s]
[INFO] Flume legacy Thrift Source ........................ SUCCESS [1.634s]
[INFO] Flume NG Clients .................................. SUCCESS [0.538s]
[INFO] Flume NG Log4j Appender ........................... SUCCESS [2.833s]
[INFO] Flume NG distribution ............................. SUCCESS [22.601s]
[INFO] Flume NG Integration Tests ........................ *FAILURE*
 [27.408s]
[INFO]
------------------------------------------------------------------------

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Mike Percy <mp...@apache.org>.
BTW I'm +1 on pulling it into 1.3.0

On Mon, Nov 19, 2012 at 12:22 AM, Mike Percy <mp...@apache.org> wrote:

> I just committed FLUME-1660 to trunk and 1.4. If Brock is cool with it
> then let's bring it into 1.3.0.
>
> Regards
> Mike
>
>
> On Sun, Nov 18, 2012 at 7:33 PM, Juhani Connolly <
> juhani_connolly@cyberagent.co.jp> wrote:
>
>> If we're doing another RC, I'd like it if we could squeeze in FLUME-1660
>> too as I've seen several users inquiring about closing tmp files, and I
>> think it provides the best solution for many of them. I believe it was
>>  bumped out of 1.3 due to time constraints but it looks like that shouldn't
>> be an issue now?
>>
>> Personally, +0 on the release, haven't found the time to do a thorough
>> check.
>>
>>
>> On 11/19/2012 01:36 AM, Jarek Jarcec Cecho wrote:
>>
>>> Thank you sir,
>>> greatly appreciated!
>>>
>>> Jarcec
>>>
>>> On Sat, Nov 17, 2012 at 07:39:57PM -0800, Mike Percy wrote:
>>>
>>>> Done.
>>>>
>>>> Regards
>>>> Mike
>>>>
>>>> On Sat, Nov 17, 2012 at 7:27 PM, Jarek Jarcec Cecho <jarcec@apache.org
>>>> >wrote:
>>>>
>>>>  Might I humbly ask for review & commit FLUME-1719 for next RC?
>>>>>
>>>>> Jarcec
>>>>>
>>>>> On Sat, Nov 17, 2012 at 07:13:26PM -0800, Hari Shreedharan wrote:
>>>>>
>>>>>> Done! Thanks Mike, Brock!
>>>>>>
>>>>>>
>>>>>> Hari
>>>>>>
>>>>>> --
>>>>>> Hari Shreedharan
>>>>>>
>>>>>>
>>>>>> On Saturday, November 17, 2012 at 7:05 PM, Brock Noland wrote:
>>>>>>
>>>>>>  Ok, let's do another rc since the same lobs with different versions
>>>>>>> is
>>>>>>>
>>>>>> probably bad.
>>>>>
>>>>>> Also I think it's fine to commit to 1.3 without another rc since we
>>>>>>>
>>>>>> are voting on a tag, not the branch.
>>>>>
>>>>>> --
>>>>>>> Brock Noland
>>>>>>> Sent with Sparrow (http://www.sparrowmailapp.**com/?sig<http://www.sparrowmailapp.com/?sig>
>>>>>>> )
>>>>>>>
>>>>>>>
>>>>>>> On Saturday, November 17, 2012 at 8:45 PM, Mike Percy wrote:
>>>>>>>
>>>>>>>  Hari,
>>>>>>>> You're correct that a -1 vote does not veto a release. A savvy user
>>>>>>>>
>>>>>>> could
>>>>>
>>>>>> delete one of the jars before deploying to production, but why make
>>>>>>>>
>>>>>>> them do
>>>>>
>>>>>> that.
>>>>>>>>
>>>>>>>> Thanks for the commit.
>>>>>>>>
>>>>>>>> Regards,
>>>>>>>> Mike
>>>>>>>>
>>>>>>>> On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <
>>>>>>>>
>>>>>>> hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.**com<hs...@cloudera.com>
>>>>> )
>>>>>
>>>>>>  wrote:
>>>>>>>>>
>>>>>>>>
>>>>>>>>
>>>>>>>>  Mike,
>>>>>>>>>
>>>>>>>>> I committed your patch for flume-1.4, and trunk.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Brock: As I posted on the jira, if you want it committed to
>>>>>>>>>
>>>>>>>> flume-1.3. the
>>>>>
>>>>>>  RM should decide if another RC is required (looks like there are 3
>>>>>>>>>
>>>>>>>> PMC +1
>>>>>
>>>>>>  already, unless someone withdraws the +1). If required, please
>>>>>>>>>
>>>>>>>> cherry-pick
>>>>>
>>>>>>  the patch to flume-1.3.0 and roll another RC.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> Thanks,
>>>>>>>>> Hari
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>> Hari Shreedharan
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
>>>>>>>>>
>>>>>>>>>  When I was reviewing the libs I noticed that netty is in there
>>>>>>>>>>
>>>>>>>>> twice
>>>>>
>>>>>>  with 2
>>>>>>>>>
>>>>>>>>>> different versions. I don't think we should ship like that.
>>>>>>>>>>
>>>>>>>>>> So, -1 from me on this RC.
>>>>>>>>>>
>>>>>>>>>> Details and a patch are here:
>>>>>>>>>> https://issues.apache.org/**jira/browse/FLUME-1723<https://issues.apache.org/jira/browse/FLUME-1723>
>>>>>>>>>>
>>>>>>>>>> Regards,
>>>>>>>>>> Mike
>>>>>>>>>>
>>>>>>>>>> On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <
>>>>>>>>>>
>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)(**mailto:
>>>>>
>>>>>>  brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
>>>>>>>>>
>>>>>>>>>> Thanks Hari, I have created FLUME-1722 for this purpose.
>>>>>>>>>>>
>>>>>>>>>>> https://issues.apache.org/**jira/browse/FLUME-1722<https://issues.apache.org/jira/browse/FLUME-1722>
>>>>>>>>>>>
>>>>>>>>>>> Brock
>>>>>>>>>>>
>>>>>>>>>>> On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
>>>>>>>>>>> <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.**com<hs...@cloudera.com>
>>>>>>>>>>> )>
>>>>>>>>>>>
>>>>>>>>>> wrote:
>>>>>
>>>>>>  +1.
>>>>>>>>>>>>
>>>>>>>>>>>> Here is what I checked:
>>>>>>>>>>>> 1. Checksums and signatures look good.
>>>>>>>>>>>> 2. Verified license file, readme, change log and notice file
>>>>>>>>>>>>
>>>>>>>>>>> (more on
>>>>>
>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> this below).
>>>>>>>>>>>
>>>>>>>>>>>> 3. Built, ran unit tests, ran an agent with the sample
>>>>>>>>>>>>
>>>>>>>>>>> config - looks
>>>>>
>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> good.
>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> Additional notes:
>>>>>>>>>>>> The LICENSE file seems to list three libraries which we do
>>>>>>>>>>>>
>>>>>>>>>>> not pull
>>>>>
>>>>>>
>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>> in:
>>>>>>>>>
>>>>>>>>>> commons logging
>>>>>>>>>>>> http-client
>>>>>>>>>>>> http-core
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> We probably should remove these from the license file too.
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> Thanks,
>>>>>>>>>>>> Hari
>>>>>>>>>>>> --
>>>>>>>>>>>> Hari Shreedharan
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> On Saturday, November 17, 2012 at 12:57 AM, Alexander
>>>>>>>>>>>>
>>>>>>>>>>> Alten-Lorenz
>>>>>
>>>>>>  wrote:
>>>>>>>>>
>>>>>>>>>>  +1
>>>>>>>>>>>>>
>>>>>>>>>>>>> - Alex
>>>>>>>>>>>>>
>>>>>>>>>>>>> On Nov 17, 2012, at 4:10 AM, Will McQueen <
>>>>>>>>>>>>>
>>>>>>>>>>>> will@cloudera.com (mailto:will@cloudera.com)(**mailto:
>>>>>
>>>>>>  will@cloudera.com (mailto:will@cloudera.com)) (mailto:
>>>>>>>>>
>>>>>>>>>> will@cloudera.com (mailto:will@cloudera.com))> wrote:
>>>>>>>>>>>
>>>>>>>>>>>> +1 on the release
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> On Nov 16, 2012, at 6:56 PM, Brock Noland <
>>>>>>>>>>>>>>
>>>>>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)(**mailto:
>>>>>
>>>>>>  brock@cloudera.com (mailto:brock@cloudera.com))(**mailto:
>>>>>>>>>
>>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
>>>>>>>>>>>
>>>>>>>>>>>>  Thanks Jarek, I don't think it's a blocker but we
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>> should
>>>>>
>>>>>>  definitely
>>>>>>>>>
>>>>>>>>>> fix it.
>>>>>>>>>>>
>>>>>>>>>>>>  FWIW, hadoop's LICENSE.txt doesn't even list the
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>> licenses of
>>>>>
>>>>>>  binaries
>>>>>>>>>>>
>>>>>>>>>>>>  and Crunch was just released with many defects to this
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>> file.
>>>>>
>>>>>>   On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>> jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
>>>>>>>>>>>
>>>>>>>>>>>>  I've noticed that LICENSE file contains entry
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> protobuf-<version>.jar whereas our binary artifacts seems to
>>>>>>>>>>>
>>>>>>>>>> be pulling
>>>>>
>>>>>>  protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do
>>>>>>>>>>>
>>>>>>>>>> need seems
>>>>>
>>>>>>
>>>>>>>>> as a
>>>>>>>>>
>>>>>>>>>> blocker to me, but it would be great if someone else with more
>>>>>>>>>>>
>>>>>>>>>> deeper
>>>>>
>>>>>>  licensing knowledge would confirm that. As long as this is
>>>>>>>>>>>
>>>>>>>>>> fine, I'm
>>>>>
>>>>>>
>>>>>>>>> +1 on
>>>>>>>>>
>>>>>>>>>> the release.
>>>>>>>>>>>
>>>>>>>>>>>>  Checked:
>>>>>>>>>>>>>>>> * Checksums, signatures seems fine
>>>>>>>>>>>>>>>> * "mvn clean compile test verify package" passes for
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> source
>>>>>
>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> artifact
>>>>>>>>>>>
>>>>>>>>>>>>  * "mvn verify" passes for binary artifact
>>>>>>>>>>>>>>>> * CHANGELOG, NOTICE, README, DEVNOTES files seems
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> fine to me
>>>>>
>>>>>>   Notes:
>>>>>>>>>>>>>>>> * Filled FLUME-1719: Example export command in
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> README do not
>>>>>
>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> properly close the string
>>>>>>>>>>>
>>>>>>>>>>>>  * Filled FLUME-1720: LICENSE file contain entry for
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> protobuf-<version>.jar, however proper artifact name is
>>>>>>>>>>> protobuf-java-<version>.jar
>>>>>>>>>>>
>>>>>>>>>>>>  Jarcec
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> Links:
>>>>>>>>>>>>>>>> 1: https://issues.apache.org/**jira/browse/FLUME-1720<https://issues.apache.org/jira/browse/FLUME-1720>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Alten-Lorenz
>>>>>>>>>
>>>>>>>>>> wrote:
>>>>>>>>>>>
>>>>>>>>>>>>  looks good to me, thanks.
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> +1
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> cheers,
>>>>>>>>>>>>>>>>> Alex
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> On Nov 16, 2012, at 12:21 AM, Brock Noland <
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)(**mailto:
>>>>>>>>>
>>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
>>>>>>>>>>>
>>>>>>>>>>>>  +1 verified sigs, hashes, built docs, and
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> verified smoke
>>>>>
>>>>>>  test
>>>>>>>>>
>>>>>>>>>>  On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
>>>>>>>>>>>
>>>>>>>>>>>>  This is the second release for Apache Flume as
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> a
>>>>>
>>>>>>
>>>>>>>>>>>
>>>>>>>>> top-level
>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> project,
>>>>>>>>>>>
>>>>>>>>>>>>  version 1.3.0. We are voting on release
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> candidate rc3.
>>>>>
>>>>>>   *** Please cast your vote within the next 72
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> hours ***
>>>>>
>>>>>>   The list of fixed issues:
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> https://git-wip-us.apache.org/**
>>>>> repos/asf?p=flume.git;a=blob_**plain;f=CHANGELOG;hb=**
>>>>> 656e049981df62316e0a8173820872**97209bf2f3<https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3>
>>>>>
>>>>>>   The tarball (*.tar.gz), signature (*.asc), and
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> checksums
>>>>>>>>>
>>>>>>>>>> (*.md5, *.sha1)
>>>>>>>>>>>
>>>>>>>>>>>>  for the source and binary artifacts can be
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> found at:
>>>>> http://people.apache.org/~**brock/flume-1.3.0-rc3/<http://people.apache.org/~brock/flume-1.3.0-rc3/>
>>>>>
>>>>>>   (Note: attempting to deploy the artifacts to
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> Nexus
>>>>>
>>>>>>  appears
>>>>>>>>>
>>>>>>>>>> to have
>>>>>>>>>>>
>>>>>>>>>>>>  locked me out of Apache LDAP so you'll have to
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> use the
>>>>>
>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> artifacts from
>>>>>>>>>>>
>>>>>>>>>>>>  the above link or via the source tag below.)
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>> The tag to be voted on:
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> https://git-wip-us.apache.org/**repos/asf?p=flume.git;a=*
>>>>> *commit;h=**656e049981df62316e0a8173820872**97209bf2f3<https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3>
>>>>>
>>>>>>   The KEYS file can be found here:
>>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>>  https://dist.apache.org/repos/**dist/release/flume/KEYS<https://dist.apache.org/repos/dist/release/flume/KEYS>
>>>>>
>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>>>>> Apache MRUnit - Unit testing MapReduce -
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> http://incubator.apache.org/**mrunit/<http://incubator.apache.org/mrunit/>
>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>>>> Alexander Alten-Lorenz
>>>>>>>>>>>>>>>>> http://mapredit.blogspot.com
>>>>>>>>>>>>>>>>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>> Apache MRUnit - Unit testing MapReduce -
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> http://incubator.apache.org/**mrunit/<http://incubator.apache.org/mrunit/>
>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> --
>>>>>>>>>>>>> Alexander Alten-Lorenz
>>>>>>>>>>>>> http://mapredit.blogspot.com
>>>>>>>>>>>>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> --
>>>>>>>>>>> Apache MRUnit - Unit testing MapReduce -
>>>>>>>>>>> http://incubator.apache.org/**mrunit/<http://incubator.apache.org/mrunit/>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>
>>
>

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Mike Percy <mp...@apache.org>.
I just committed FLUME-1660 to trunk and 1.4. If Brock is cool with it then
let's bring it into 1.3.0.

Regards
Mike

On Sun, Nov 18, 2012 at 7:33 PM, Juhani Connolly <
juhani_connolly@cyberagent.co.jp> wrote:

> If we're doing another RC, I'd like it if we could squeeze in FLUME-1660
> too as I've seen several users inquiring about closing tmp files, and I
> think it provides the best solution for many of them. I believe it was
>  bumped out of 1.3 due to time constraints but it looks like that shouldn't
> be an issue now?
>
> Personally, +0 on the release, haven't found the time to do a thorough
> check.
>
>
> On 11/19/2012 01:36 AM, Jarek Jarcec Cecho wrote:
>
>> Thank you sir,
>> greatly appreciated!
>>
>> Jarcec
>>
>> On Sat, Nov 17, 2012 at 07:39:57PM -0800, Mike Percy wrote:
>>
>>> Done.
>>>
>>> Regards
>>> Mike
>>>
>>> On Sat, Nov 17, 2012 at 7:27 PM, Jarek Jarcec Cecho <jarcec@apache.org
>>> >wrote:
>>>
>>>  Might I humbly ask for review & commit FLUME-1719 for next RC?
>>>>
>>>> Jarcec
>>>>
>>>> On Sat, Nov 17, 2012 at 07:13:26PM -0800, Hari Shreedharan wrote:
>>>>
>>>>> Done! Thanks Mike, Brock!
>>>>>
>>>>>
>>>>> Hari
>>>>>
>>>>> --
>>>>> Hari Shreedharan
>>>>>
>>>>>
>>>>> On Saturday, November 17, 2012 at 7:05 PM, Brock Noland wrote:
>>>>>
>>>>>  Ok, let's do another rc since the same lobs with different versions is
>>>>>>
>>>>> probably bad.
>>>>
>>>>> Also I think it's fine to commit to 1.3 without another rc since we
>>>>>>
>>>>> are voting on a tag, not the branch.
>>>>
>>>>> --
>>>>>> Brock Noland
>>>>>> Sent with Sparrow (http://www.sparrowmailapp.**com/?sig<http://www.sparrowmailapp.com/?sig>
>>>>>> )
>>>>>>
>>>>>>
>>>>>> On Saturday, November 17, 2012 at 8:45 PM, Mike Percy wrote:
>>>>>>
>>>>>>  Hari,
>>>>>>> You're correct that a -1 vote does not veto a release. A savvy user
>>>>>>>
>>>>>> could
>>>>
>>>>> delete one of the jars before deploying to production, but why make
>>>>>>>
>>>>>> them do
>>>>
>>>>> that.
>>>>>>>
>>>>>>> Thanks for the commit.
>>>>>>>
>>>>>>> Regards,
>>>>>>> Mike
>>>>>>>
>>>>>>> On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <
>>>>>>>
>>>>>> hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.**com<hs...@cloudera.com>
>>>> )
>>>>
>>>>>  wrote:
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>>>  Mike,
>>>>>>>>
>>>>>>>> I committed your patch for flume-1.4, and trunk.
>>>>>>>>
>>>>>>>>
>>>>>>>> Brock: As I posted on the jira, if you want it committed to
>>>>>>>>
>>>>>>> flume-1.3. the
>>>>
>>>>>  RM should decide if another RC is required (looks like there are 3
>>>>>>>>
>>>>>>> PMC +1
>>>>
>>>>>  already, unless someone withdraws the +1). If required, please
>>>>>>>>
>>>>>>> cherry-pick
>>>>
>>>>>  the patch to flume-1.3.0 and roll another RC.
>>>>>>>>
>>>>>>>>
>>>>>>>> Thanks,
>>>>>>>> Hari
>>>>>>>>
>>>>>>>> --
>>>>>>>> Hari Shreedharan
>>>>>>>>
>>>>>>>>
>>>>>>>> On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
>>>>>>>>
>>>>>>>>  When I was reviewing the libs I noticed that netty is in there
>>>>>>>>>
>>>>>>>> twice
>>>>
>>>>>  with 2
>>>>>>>>
>>>>>>>>> different versions. I don't think we should ship like that.
>>>>>>>>>
>>>>>>>>> So, -1 from me on this RC.
>>>>>>>>>
>>>>>>>>> Details and a patch are here:
>>>>>>>>> https://issues.apache.org/**jira/browse/FLUME-1723<https://issues.apache.org/jira/browse/FLUME-1723>
>>>>>>>>>
>>>>>>>>> Regards,
>>>>>>>>> Mike
>>>>>>>>>
>>>>>>>>> On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <
>>>>>>>>>
>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)(**mailto:
>>>>
>>>>>  brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
>>>>>>>>
>>>>>>>>> Thanks Hari, I have created FLUME-1722 for this purpose.
>>>>>>>>>>
>>>>>>>>>> https://issues.apache.org/**jira/browse/FLUME-1722<https://issues.apache.org/jira/browse/FLUME-1722>
>>>>>>>>>>
>>>>>>>>>> Brock
>>>>>>>>>>
>>>>>>>>>> On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
>>>>>>>>>> <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.**com<hs...@cloudera.com>
>>>>>>>>>> )>
>>>>>>>>>>
>>>>>>>>> wrote:
>>>>
>>>>>  +1.
>>>>>>>>>>>
>>>>>>>>>>> Here is what I checked:
>>>>>>>>>>> 1. Checksums and signatures look good.
>>>>>>>>>>> 2. Verified license file, readme, change log and notice file
>>>>>>>>>>>
>>>>>>>>>> (more on
>>>>
>>>>>
>>>>>>>>>>
>>>>>>>>>> this below).
>>>>>>>>>>
>>>>>>>>>>> 3. Built, ran unit tests, ran an agent with the sample
>>>>>>>>>>>
>>>>>>>>>> config - looks
>>>>
>>>>>
>>>>>>>>>>
>>>>>>>>>> good.
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Additional notes:
>>>>>>>>>>> The LICENSE file seems to list three libraries which we do
>>>>>>>>>>>
>>>>>>>>>> not pull
>>>>
>>>>>
>>>>>>>>>>
>>>>>>>>>
>>>>>>>> in:
>>>>>>>>
>>>>>>>>> commons logging
>>>>>>>>>>> http-client
>>>>>>>>>>> http-core
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> We probably should remove these from the license file too.
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> Thanks,
>>>>>>>>>>> Hari
>>>>>>>>>>> --
>>>>>>>>>>> Hari Shreedharan
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> On Saturday, November 17, 2012 at 12:57 AM, Alexander
>>>>>>>>>>>
>>>>>>>>>> Alten-Lorenz
>>>>
>>>>>  wrote:
>>>>>>>>
>>>>>>>>>  +1
>>>>>>>>>>>>
>>>>>>>>>>>> - Alex
>>>>>>>>>>>>
>>>>>>>>>>>> On Nov 17, 2012, at 4:10 AM, Will McQueen <
>>>>>>>>>>>>
>>>>>>>>>>> will@cloudera.com (mailto:will@cloudera.com)(**mailto:
>>>>
>>>>>  will@cloudera.com (mailto:will@cloudera.com)) (mailto:
>>>>>>>>
>>>>>>>>> will@cloudera.com (mailto:will@cloudera.com))> wrote:
>>>>>>>>>>
>>>>>>>>>>> +1 on the release
>>>>>>>>>>>>>
>>>>>>>>>>>>> On Nov 16, 2012, at 6:56 PM, Brock Noland <
>>>>>>>>>>>>>
>>>>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)(**mailto:
>>>>
>>>>>  brock@cloudera.com (mailto:brock@cloudera.com))(**mailto:
>>>>>>>>
>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
>>>>>>>>>>
>>>>>>>>>>>  Thanks Jarek, I don't think it's a blocker but we
>>>>>>>>>>>>>>
>>>>>>>>>>>>> should
>>>>
>>>>>  definitely
>>>>>>>>
>>>>>>>>> fix it.
>>>>>>>>>>
>>>>>>>>>>>  FWIW, hadoop's LICENSE.txt doesn't even list the
>>>>>>>>>>>>>>
>>>>>>>>>>>>> licenses of
>>>>
>>>>>  binaries
>>>>>>>>>>
>>>>>>>>>>>  and Crunch was just released with many defects to this
>>>>>>>>>>>>>>
>>>>>>>>>>>>> file.
>>>>
>>>>>   On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
>>>>>>>>>>>>>>
>>>>>>>>>>>>> jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
>>>>>>>>>>
>>>>>>>>>>>  I've noticed that LICENSE file contains entry
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> protobuf-<version>.jar whereas our binary artifacts seems to
>>>>>>>>>>
>>>>>>>>> be pulling
>>>>
>>>>>  protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do
>>>>>>>>>>
>>>>>>>>> need seems
>>>>
>>>>>
>>>>>>>> as a
>>>>>>>>
>>>>>>>>> blocker to me, but it would be great if someone else with more
>>>>>>>>>>
>>>>>>>>> deeper
>>>>
>>>>>  licensing knowledge would confirm that. As long as this is
>>>>>>>>>>
>>>>>>>>> fine, I'm
>>>>
>>>>>
>>>>>>>> +1 on
>>>>>>>>
>>>>>>>>> the release.
>>>>>>>>>>
>>>>>>>>>>>  Checked:
>>>>>>>>>>>>>>> * Checksums, signatures seems fine
>>>>>>>>>>>>>>> * "mvn clean compile test verify package" passes for
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>> source
>>>>
>>>>>
>>>>>>>>>>
>>>>>>>>>> artifact
>>>>>>>>>>
>>>>>>>>>>>  * "mvn verify" passes for binary artifact
>>>>>>>>>>>>>>> * CHANGELOG, NOTICE, README, DEVNOTES files seems
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>> fine to me
>>>>
>>>>>   Notes:
>>>>>>>>>>>>>>> * Filled FLUME-1719: Example export command in
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>> README do not
>>>>
>>>>>
>>>>>>>>>>
>>>>>>>>>> properly close the string
>>>>>>>>>>
>>>>>>>>>>>  * Filled FLUME-1720: LICENSE file contain entry for
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> protobuf-<version>.jar, however proper artifact name is
>>>>>>>>>> protobuf-java-<version>.jar
>>>>>>>>>>
>>>>>>>>>>>  Jarcec
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> Links:
>>>>>>>>>>>>>>> 1: https://issues.apache.org/**jira/browse/FLUME-1720<https://issues.apache.org/jira/browse/FLUME-1720>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Alten-Lorenz
>>>>>>>>
>>>>>>>>> wrote:
>>>>>>>>>>
>>>>>>>>>>>  looks good to me, thanks.
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> +1
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> cheers,
>>>>>>>>>>>>>>>> Alex
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> On Nov 16, 2012, at 12:21 AM, Brock Noland <
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)(**mailto:
>>>>>>>>
>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
>>>>>>>>>>
>>>>>>>>>>>  +1 verified sigs, hashes, built docs, and
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> verified smoke
>>>>
>>>>>  test
>>>>>>>>
>>>>>>>>>  On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
>>>>>>>>>>
>>>>>>>>>>>  This is the second release for Apache Flume as
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> a
>>>>
>>>>>
>>>>>>>>>>
>>>>>>>> top-level
>>>>>>>>
>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> project,
>>>>>>>>>>
>>>>>>>>>>>  version 1.3.0. We are voting on release
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> candidate rc3.
>>>>
>>>>>   *** Please cast your vote within the next 72
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> hours ***
>>>>
>>>>>   The list of fixed issues:
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> https://git-wip-us.apache.org/**
>>>> repos/asf?p=flume.git;a=blob_**plain;f=CHANGELOG;hb=**
>>>> 656e049981df62316e0a8173820872**97209bf2f3<https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3>
>>>>
>>>>>   The tarball (*.tar.gz), signature (*.asc), and
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> checksums
>>>>>>>>
>>>>>>>>> (*.md5, *.sha1)
>>>>>>>>>>
>>>>>>>>>>>  for the source and binary artifacts can be
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> found at:
>>>> http://people.apache.org/~**brock/flume-1.3.0-rc3/<http://people.apache.org/~brock/flume-1.3.0-rc3/>
>>>>
>>>>>   (Note: attempting to deploy the artifacts to
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> Nexus
>>>>
>>>>>  appears
>>>>>>>>
>>>>>>>>> to have
>>>>>>>>>>
>>>>>>>>>>>  locked me out of Apache LDAP so you'll have to
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> use the
>>>>
>>>>>
>>>>>>>>>>
>>>>>>>>>> artifacts from
>>>>>>>>>>
>>>>>>>>>>>  the above link or via the source tag below.)
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>> The tag to be voted on:
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> https://git-wip-us.apache.org/**repos/asf?p=flume.git;a=**
>>>> commit;h=**656e049981df62316e0a8173820872**97209bf2f3<https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3>
>>>>
>>>>>   The KEYS file can be found here:
>>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>>  https://dist.apache.org/repos/**dist/release/flume/KEYS<https://dist.apache.org/repos/dist/release/flume/KEYS>
>>>>
>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>>>> Apache MRUnit - Unit testing MapReduce -
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> http://incubator.apache.org/**mrunit/<http://incubator.apache.org/mrunit/>
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>>> Alexander Alten-Lorenz
>>>>>>>>>>>>>>>> http://mapredit.blogspot.com
>>>>>>>>>>>>>>>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> --
>>>>>>>>>>>>>> Apache MRUnit - Unit testing MapReduce -
>>>>>>>>>>>>>>
>>>>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> http://incubator.apache.org/**mrunit/<http://incubator.apache.org/mrunit/>
>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>>> --
>>>>>>>>>>>> Alexander Alten-Lorenz
>>>>>>>>>>>> http://mapredit.blogspot.com
>>>>>>>>>>>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>>>>>>>>>>>>
>>>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> --
>>>>>>>>>> Apache MRUnit - Unit testing MapReduce -
>>>>>>>>>> http://incubator.apache.org/**mrunit/<http://incubator.apache.org/mrunit/>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>
>>>>>>
>>>>>
>

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Juhani Connolly <ju...@cyberagent.co.jp>.
If we're doing another RC, I'd like it if we could squeeze in FLUME-1660 
too as I've seen several users inquiring about closing tmp files, and I 
think it provides the best solution for many of them. I believe it was  
bumped out of 1.3 due to time constraints but it looks like that 
shouldn't be an issue now?

Personally, +0 on the release, haven't found the time to do a thorough 
check.

On 11/19/2012 01:36 AM, Jarek Jarcec Cecho wrote:
> Thank you sir,
> greatly appreciated!
>
> Jarcec
>
> On Sat, Nov 17, 2012 at 07:39:57PM -0800, Mike Percy wrote:
>> Done.
>>
>> Regards
>> Mike
>>
>> On Sat, Nov 17, 2012 at 7:27 PM, Jarek Jarcec Cecho <ja...@apache.org>wrote:
>>
>>> Might I humbly ask for review & commit FLUME-1719 for next RC?
>>>
>>> Jarcec
>>>
>>> On Sat, Nov 17, 2012 at 07:13:26PM -0800, Hari Shreedharan wrote:
>>>> Done! Thanks Mike, Brock!
>>>>
>>>>
>>>> Hari
>>>>
>>>> --
>>>> Hari Shreedharan
>>>>
>>>>
>>>> On Saturday, November 17, 2012 at 7:05 PM, Brock Noland wrote:
>>>>
>>>>> Ok, let's do another rc since the same lobs with different versions is
>>> probably bad.
>>>>> Also I think it's fine to commit to 1.3 without another rc since we
>>> are voting on a tag, not the branch.
>>>>> --
>>>>> Brock Noland
>>>>> Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
>>>>>
>>>>>
>>>>> On Saturday, November 17, 2012 at 8:45 PM, Mike Percy wrote:
>>>>>
>>>>>> Hari,
>>>>>> You're correct that a -1 vote does not veto a release. A savvy user
>>> could
>>>>>> delete one of the jars before deploying to production, but why make
>>> them do
>>>>>> that.
>>>>>>
>>>>>> Thanks for the commit.
>>>>>>
>>>>>> Regards,
>>>>>> Mike
>>>>>>
>>>>>> On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <
>>> hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)
>>>>>>> wrote:
>>>>>>
>>>>>>
>>>>>>> Mike,
>>>>>>>
>>>>>>> I committed your patch for flume-1.4, and trunk.
>>>>>>>
>>>>>>>
>>>>>>> Brock: As I posted on the jira, if you want it committed to
>>> flume-1.3. the
>>>>>>> RM should decide if another RC is required (looks like there are 3
>>> PMC +1
>>>>>>> already, unless someone withdraws the +1). If required, please
>>> cherry-pick
>>>>>>> the patch to flume-1.3.0 and roll another RC.
>>>>>>>
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Hari
>>>>>>>
>>>>>>> --
>>>>>>> Hari Shreedharan
>>>>>>>
>>>>>>>
>>>>>>> On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
>>>>>>>
>>>>>>>> When I was reviewing the libs I noticed that netty is in there
>>> twice
>>>>>>> with 2
>>>>>>>> different versions. I don't think we should ship like that.
>>>>>>>>
>>>>>>>> So, -1 from me on this RC.
>>>>>>>>
>>>>>>>> Details and a patch are here:
>>>>>>>> https://issues.apache.org/jira/browse/FLUME-1723
>>>>>>>>
>>>>>>>> Regards,
>>>>>>>> Mike
>>>>>>>>
>>>>>>>> On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <
>>> brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
>>>>>>>>> Thanks Hari, I have created FLUME-1722 for this purpose.
>>>>>>>>>
>>>>>>>>> https://issues.apache.org/jira/browse/FLUME-1722
>>>>>>>>>
>>>>>>>>> Brock
>>>>>>>>>
>>>>>>>>> On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
>>>>>>>>> <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)>
>>> wrote:
>>>>>>>>>> +1.
>>>>>>>>>>
>>>>>>>>>> Here is what I checked:
>>>>>>>>>> 1. Checksums and signatures look good.
>>>>>>>>>> 2. Verified license file, readme, change log and notice file
>>> (more on
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> this below).
>>>>>>>>>> 3. Built, ran unit tests, ran an agent with the sample
>>> config - looks
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> good.
>>>>>>>>>>
>>>>>>>>>> Additional notes:
>>>>>>>>>> The LICENSE file seems to list three libraries which we do
>>> not pull
>>>>>>>>>
>>>>>>>>
>>>>>>>
>>>>>>> in:
>>>>>>>>>> commons logging
>>>>>>>>>> http-client
>>>>>>>>>> http-core
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> We probably should remove these from the license file too.
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> Thanks,
>>>>>>>>>> Hari
>>>>>>>>>> --
>>>>>>>>>> Hari Shreedharan
>>>>>>>>>>
>>>>>>>>>>
>>>>>>>>>> On Saturday, November 17, 2012 at 12:57 AM, Alexander
>>> Alten-Lorenz
>>>>>>> wrote:
>>>>>>>>>>> +1
>>>>>>>>>>>
>>>>>>>>>>> - Alex
>>>>>>>>>>>
>>>>>>>>>>> On Nov 17, 2012, at 4:10 AM, Will McQueen <
>>> will@cloudera.com (mailto:will@cloudera.com)(mailto:
>>>>>>> will@cloudera.com (mailto:will@cloudera.com)) (mailto:
>>>>>>>>> will@cloudera.com (mailto:will@cloudera.com))> wrote:
>>>>>>>>>>>> +1 on the release
>>>>>>>>>>>>
>>>>>>>>>>>> On Nov 16, 2012, at 6:56 PM, Brock Noland <
>>> brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com))(mailto:
>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
>>>>>>>>>>>>> Thanks Jarek, I don't think it's a blocker but we
>>> should
>>>>>>> definitely
>>>>>>>>> fix it.
>>>>>>>>>>>>> FWIW, hadoop's LICENSE.txt doesn't even list the
>>> licenses of
>>>>>>>>> binaries
>>>>>>>>>>>>> and Crunch was just released with many defects to this
>>> file.
>>>>>>>>>>>>> On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
>>>>>>>>> jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
>>>>>>>>>>>>>> I've noticed that LICENSE file contains entry
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> protobuf-<version>.jar whereas our binary artifacts seems to
>>> be pulling
>>>>>>>>> protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do
>>> need seems
>>>>>>>
>>>>>>> as a
>>>>>>>>> blocker to me, but it would be great if someone else with more
>>> deeper
>>>>>>>>> licensing knowledge would confirm that. As long as this is
>>> fine, I'm
>>>>>>>
>>>>>>> +1 on
>>>>>>>>> the release.
>>>>>>>>>>>>>> Checked:
>>>>>>>>>>>>>> * Checksums, signatures seems fine
>>>>>>>>>>>>>> * "mvn clean compile test verify package" passes for
>>> source
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> artifact
>>>>>>>>>>>>>> * "mvn verify" passes for binary artifact
>>>>>>>>>>>>>> * CHANGELOG, NOTICE, README, DEVNOTES files seems
>>> fine to me
>>>>>>>>>>>>>> Notes:
>>>>>>>>>>>>>> * Filled FLUME-1719: Example export command in
>>> README do not
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> properly close the string
>>>>>>>>>>>>>> * Filled FLUME-1720: LICENSE file contain entry for
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> protobuf-<version>.jar, however proper artifact name is
>>>>>>>>> protobuf-java-<version>.jar
>>>>>>>>>>>>>> Jarcec
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> Links:
>>>>>>>>>>>>>> 1: https://issues.apache.org/jira/browse/FLUME-1720
>>>>>>>>>>>>>>
>>>>>>>>>>>>>> On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
>>>>>>> Alten-Lorenz
>>>>>>>>> wrote:
>>>>>>>>>>>>>>> looks good to me, thanks.
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> +1
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> cheers,
>>>>>>>>>>>>>>> Alex
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> On Nov 16, 2012, at 12:21 AM, Brock Noland <
>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
>>>>>>>>>>>>>>>> +1 verified sigs, hashes, built docs, and
>>> verified smoke
>>>>>>> test
>>>>>>>>>>>>>>>> On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
>>>>>>>>> brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
>>>>>>>>>>>>>>>>> This is the second release for Apache Flume as
>>> a
>>>>>>>>>
>>>>>>>
>>>>>>> top-level
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> project,
>>>>>>>>>>>>>>>>> version 1.3.0. We are voting on release
>>> candidate rc3.
>>>>>>>>>>>>>>>>> *** Please cast your vote within the next 72
>>> hours ***
>>>>>>>>>>>>>>>>> The list of fixed issues:
>>> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
>>>>>>>>>>>>>>>>> The tarball (*.tar.gz), signature (*.asc), and
>>>>>>> checksums
>>>>>>>>> (*.md5, *.sha1)
>>>>>>>>>>>>>>>>> for the source and binary artifacts can be
>>> found at:
>>> http://people.apache.org/~brock/flume-1.3.0-rc3/
>>>>>>>>>>>>>>>>> (Note: attempting to deploy the artifacts to
>>> Nexus
>>>>>>> appears
>>>>>>>>> to have
>>>>>>>>>>>>>>>>> locked me out of Apache LDAP so you'll have to
>>> use the
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> artifacts from
>>>>>>>>>>>>>>>>> the above link or via the source tag below.)
>>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>> The tag to be voted on:
>>> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
>>>>>>>>>>>>>>>>> The KEYS file can be found here:
>>>>>>>>>>>>>>>>>
>>> https://dist.apache.org/repos/dist/release/flume/KEYS
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>>> Apache MRUnit - Unit testing MapReduce -
>>>>>>>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> http://incubator.apache.org/mrunit/
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>>> --
>>>>>>>>>>>>>>> Alexander Alten-Lorenz
>>>>>>>>>>>>>>> http://mapredit.blogspot.com
>>>>>>>>>>>>>>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>>>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>>
>>>>>>>>>>>>> --
>>>>>>>>>>>>> Apache MRUnit - Unit testing MapReduce -
>>>>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> http://incubator.apache.org/mrunit/
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>> --
>>>>>>>>>>> Alexander Alten-Lorenz
>>>>>>>>>>> http://mapredit.blogspot.com
>>>>>>>>>>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>>>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>> --
>>>>>>>>> Apache MRUnit - Unit testing MapReduce -
>>>>>>>>> http://incubator.apache.org/mrunit/
>>>>>>>>>
>>>>>>>
>>>>>>
>>>>>
>>>>>
>>>>


Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Jarek Jarcec Cecho <ja...@apache.org>.
Thank you sir,
greatly appreciated!

Jarcec

On Sat, Nov 17, 2012 at 07:39:57PM -0800, Mike Percy wrote:
> Done.
> 
> Regards
> Mike
> 
> On Sat, Nov 17, 2012 at 7:27 PM, Jarek Jarcec Cecho <ja...@apache.org>wrote:
> 
> > Might I humbly ask for review & commit FLUME-1719 for next RC?
> >
> > Jarcec
> >
> > On Sat, Nov 17, 2012 at 07:13:26PM -0800, Hari Shreedharan wrote:
> > > Done! Thanks Mike, Brock!
> > >
> > >
> > > Hari
> > >
> > > --
> > > Hari Shreedharan
> > >
> > >
> > > On Saturday, November 17, 2012 at 7:05 PM, Brock Noland wrote:
> > >
> > > > Ok, let's do another rc since the same lobs with different versions is
> > probably bad.
> > > >
> > > > Also I think it's fine to commit to 1.3 without another rc since we
> > are voting on a tag, not the branch.
> > > >
> > > > --
> > > > Brock Noland
> > > > Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
> > > >
> > > >
> > > > On Saturday, November 17, 2012 at 8:45 PM, Mike Percy wrote:
> > > >
> > > > > Hari,
> > > > > You're correct that a -1 vote does not veto a release. A savvy user
> > could
> > > > > delete one of the jars before deploying to production, but why make
> > them do
> > > > > that.
> > > > >
> > > > > Thanks for the commit.
> > > > >
> > > > > Regards,
> > > > > Mike
> > > > >
> > > > > On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <
> > hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)
> > > > > > wrote:
> > > > >
> > > > >
> > > > >
> > > > > > Mike,
> > > > > >
> > > > > > I committed your patch for flume-1.4, and trunk.
> > > > > >
> > > > > >
> > > > > > Brock: As I posted on the jira, if you want it committed to
> > flume-1.3. the
> > > > > > RM should decide if another RC is required (looks like there are 3
> > PMC +1
> > > > > > already, unless someone withdraws the +1). If required, please
> > cherry-pick
> > > > > > the patch to flume-1.3.0 and roll another RC.
> > > > > >
> > > > > >
> > > > > > Thanks,
> > > > > > Hari
> > > > > >
> > > > > > --
> > > > > > Hari Shreedharan
> > > > > >
> > > > > >
> > > > > > On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
> > > > > >
> > > > > > > When I was reviewing the libs I noticed that netty is in there
> > twice
> > > > > > with 2
> > > > > > > different versions. I don't think we should ship like that.
> > > > > > >
> > > > > > > So, -1 from me on this RC.
> > > > > > >
> > > > > > > Details and a patch are here:
> > > > > > > https://issues.apache.org/jira/browse/FLUME-1723
> > > > > > >
> > > > > > > Regards,
> > > > > > > Mike
> > > > > > >
> > > > > > > On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <
> > brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > >
> > > > > > > > Thanks Hari, I have created FLUME-1722 for this purpose.
> > > > > > > >
> > > > > > > > https://issues.apache.org/jira/browse/FLUME-1722
> > > > > > > >
> > > > > > > > Brock
> > > > > > > >
> > > > > > > > On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
> > > > > > > > <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)>
> > wrote:
> > > > > > > > > +1.
> > > > > > > > >
> > > > > > > > > Here is what I checked:
> > > > > > > > > 1. Checksums and signatures look good.
> > > > > > > > > 2. Verified license file, readme, change log and notice file
> > (more on
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > this below).
> > > > > > > > > 3. Built, ran unit tests, ran an agent with the sample
> > config - looks
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > good.
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > Additional notes:
> > > > > > > > > The LICENSE file seems to list three libraries which we do
> > not pull
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > > in:
> > > > > > > > >
> > > > > > > > > commons logging
> > > > > > > > > http-client
> > > > > > > > > http-core
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > We probably should remove these from the license file too.
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > Thanks,
> > > > > > > > > Hari
> > > > > > > > > --
> > > > > > > > > Hari Shreedharan
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > On Saturday, November 17, 2012 at 12:57 AM, Alexander
> > Alten-Lorenz
> > > > > > wrote:
> > > > > > > > >
> > > > > > > > > > +1
> > > > > > > > > >
> > > > > > > > > > - Alex
> > > > > > > > > >
> > > > > > > > > > On Nov 17, 2012, at 4:10 AM, Will McQueen <
> > will@cloudera.com (mailto:will@cloudera.com)(mailto:
> > > > > > will@cloudera.com (mailto:will@cloudera.com)) (mailto:
> > > > > > > > will@cloudera.com (mailto:will@cloudera.com))> wrote:
> > > > > > > > > >
> > > > > > > > > > > +1 on the release
> > > > > > > > > > >
> > > > > > > > > > > On Nov 16, 2012, at 6:56 PM, Brock Noland <
> > brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > > > brock@cloudera.com (mailto:brock@cloudera.com))(mailto:
> > > > > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > > > >
> > > > > > > > > > > > Thanks Jarek, I don't think it's a blocker but we
> > should
> > > > > > definitely
> > > > > > > > fix it.
> > > > > > > > > > > >
> > > > > > > > > > > > FWIW, hadoop's LICENSE.txt doesn't even list the
> > licenses of
> > > > > > > > binaries
> > > > > > > > > > > > and Crunch was just released with many defects to this
> > file.
> > > > > > > > > > > >
> > > > > > > > > > > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
> > > > > > > > jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> > > > > > > > > > > > > I've noticed that LICENSE file contains entry
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > protobuf-<version>.jar whereas our binary artifacts seems to
> > be pulling
> > > > > > > > protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do
> > need seems
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > > as a
> > > > > > > > blocker to me, but it would be great if someone else with more
> > deeper
> > > > > > > > licensing knowledge would confirm that. As long as this is
> > fine, I'm
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > > +1 on
> > > > > > > > the release.
> > > > > > > > > > > > >
> > > > > > > > > > > > > Checked:
> > > > > > > > > > > > > * Checksums, signatures seems fine
> > > > > > > > > > > > > * "mvn clean compile test verify package" passes for
> > source
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > artifact
> > > > > > > > > > > > > * "mvn verify" passes for binary artifact
> > > > > > > > > > > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems
> > fine to me
> > > > > > > > > > > > >
> > > > > > > > > > > > > Notes:
> > > > > > > > > > > > > * Filled FLUME-1719: Example export command in
> > README do not
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > properly close the string
> > > > > > > > > > > > > * Filled FLUME-1720: LICENSE file contain entry for
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > protobuf-<version>.jar, however proper artifact name is
> > > > > > > > protobuf-java-<version>.jar
> > > > > > > > > > > > >
> > > > > > > > > > > > > Jarcec
> > > > > > > > > > > > >
> > > > > > > > > > > > > Links:
> > > > > > > > > > > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> > > > > > > > > > > > >
> > > > > > > > > > > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
> > > > > > Alten-Lorenz
> > > > > > > > wrote:
> > > > > > > > > > > > > > looks good to me, thanks.
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > +1
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > cheers,
> > > > > > > > > > > > > > Alex
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <
> > > > > > brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > > +1 verified sigs, hashes, built docs, and
> > verified smoke
> > > > > > test
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
> > > > > > > > brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > > > > > > > > > > > > This is the second release for Apache Flume as
> > a
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > > top-level
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > project,
> > > > > > > > > > > > > > > > version 1.3.0. We are voting on release
> > candidate rc3.
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > *** Please cast your vote within the next 72
> > hours ***
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > The list of fixed issues:
> > > > > >
> > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > The tarball (*.tar.gz), signature (*.asc), and
> > > > > > checksums
> > > > > > > > (*.md5, *.sha1)
> > > > > > > > > > > > > > > > for the source and binary artifacts can be
> > found at:
> > > > > > > > > > > > > > > >
> > http://people.apache.org/~brock/flume-1.3.0-rc3/
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > (Note: attempting to deploy the artifacts to
> > Nexus
> > > > > > appears
> > > > > > > > to have
> > > > > > > > > > > > > > > > locked me out of Apache LDAP so you'll have to
> > use the
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > artifacts from
> > > > > > > > > > > > > > > > the above link or via the source tag below.)
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > The tag to be voted on:
> > > > > >
> > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > > The KEYS file can be found here:
> > > > > > > > > > > > > > > >
> > https://dist.apache.org/repos/dist/release/flume/KEYS
> > > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > --
> > > > > > > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > --
> > > > > > > > > > > > > > Alexander Alten-Lorenz
> > > > > > > > > > > > > > http://mapredit.blogspot.com
> > > > > > > > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > > --
> > > > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > --
> > > > > > > > > > Alexander Alten-Lorenz
> > > > > > > > > > http://mapredit.blogspot.com
> > > > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > --
> > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > http://incubator.apache.org/mrunit/
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > >
> > > >
> > > >
> > >
> > >
> >

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Mike Percy <mp...@apache.org>.
Done.

Regards
Mike

On Sat, Nov 17, 2012 at 7:27 PM, Jarek Jarcec Cecho <ja...@apache.org>wrote:

> Might I humbly ask for review & commit FLUME-1719 for next RC?
>
> Jarcec
>
> On Sat, Nov 17, 2012 at 07:13:26PM -0800, Hari Shreedharan wrote:
> > Done! Thanks Mike, Brock!
> >
> >
> > Hari
> >
> > --
> > Hari Shreedharan
> >
> >
> > On Saturday, November 17, 2012 at 7:05 PM, Brock Noland wrote:
> >
> > > Ok, let's do another rc since the same lobs with different versions is
> probably bad.
> > >
> > > Also I think it's fine to commit to 1.3 without another rc since we
> are voting on a tag, not the branch.
> > >
> > > --
> > > Brock Noland
> > > Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
> > >
> > >
> > > On Saturday, November 17, 2012 at 8:45 PM, Mike Percy wrote:
> > >
> > > > Hari,
> > > > You're correct that a -1 vote does not veto a release. A savvy user
> could
> > > > delete one of the jars before deploying to production, but why make
> them do
> > > > that.
> > > >
> > > > Thanks for the commit.
> > > >
> > > > Regards,
> > > > Mike
> > > >
> > > > On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <
> hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)
> > > > > wrote:
> > > >
> > > >
> > > >
> > > > > Mike,
> > > > >
> > > > > I committed your patch for flume-1.4, and trunk.
> > > > >
> > > > >
> > > > > Brock: As I posted on the jira, if you want it committed to
> flume-1.3. the
> > > > > RM should decide if another RC is required (looks like there are 3
> PMC +1
> > > > > already, unless someone withdraws the +1). If required, please
> cherry-pick
> > > > > the patch to flume-1.3.0 and roll another RC.
> > > > >
> > > > >
> > > > > Thanks,
> > > > > Hari
> > > > >
> > > > > --
> > > > > Hari Shreedharan
> > > > >
> > > > >
> > > > > On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
> > > > >
> > > > > > When I was reviewing the libs I noticed that netty is in there
> twice
> > > > > with 2
> > > > > > different versions. I don't think we should ship like that.
> > > > > >
> > > > > > So, -1 from me on this RC.
> > > > > >
> > > > > > Details and a patch are here:
> > > > > > https://issues.apache.org/jira/browse/FLUME-1723
> > > > > >
> > > > > > Regards,
> > > > > > Mike
> > > > > >
> > > > > > On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <
> brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > >
> > > > > > > Thanks Hari, I have created FLUME-1722 for this purpose.
> > > > > > >
> > > > > > > https://issues.apache.org/jira/browse/FLUME-1722
> > > > > > >
> > > > > > > Brock
> > > > > > >
> > > > > > > On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
> > > > > > > <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)>
> wrote:
> > > > > > > > +1.
> > > > > > > >
> > > > > > > > Here is what I checked:
> > > > > > > > 1. Checksums and signatures look good.
> > > > > > > > 2. Verified license file, readme, change log and notice file
> (more on
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > this below).
> > > > > > > > 3. Built, ran unit tests, ran an agent with the sample
> config - looks
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > good.
> > > > > > > >
> > > > > > > >
> > > > > > > > Additional notes:
> > > > > > > > The LICENSE file seems to list three libraries which we do
> not pull
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > in:
> > > > > > > >
> > > > > > > > commons logging
> > > > > > > > http-client
> > > > > > > > http-core
> > > > > > > >
> > > > > > > >
> > > > > > > > We probably should remove these from the license file too.
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > Thanks,
> > > > > > > > Hari
> > > > > > > > --
> > > > > > > > Hari Shreedharan
> > > > > > > >
> > > > > > > >
> > > > > > > > On Saturday, November 17, 2012 at 12:57 AM, Alexander
> Alten-Lorenz
> > > > > wrote:
> > > > > > > >
> > > > > > > > > +1
> > > > > > > > >
> > > > > > > > > - Alex
> > > > > > > > >
> > > > > > > > > On Nov 17, 2012, at 4:10 AM, Will McQueen <
> will@cloudera.com (mailto:will@cloudera.com)(mailto:
> > > > > will@cloudera.com (mailto:will@cloudera.com)) (mailto:
> > > > > > > will@cloudera.com (mailto:will@cloudera.com))> wrote:
> > > > > > > > >
> > > > > > > > > > +1 on the release
> > > > > > > > > >
> > > > > > > > > > On Nov 16, 2012, at 6:56 PM, Brock Noland <
> brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > > brock@cloudera.com (mailto:brock@cloudera.com))(mailto:
> > > > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > > >
> > > > > > > > > > > Thanks Jarek, I don't think it's a blocker but we
> should
> > > > > definitely
> > > > > > > fix it.
> > > > > > > > > > >
> > > > > > > > > > > FWIW, hadoop's LICENSE.txt doesn't even list the
> licenses of
> > > > > > > binaries
> > > > > > > > > > > and Crunch was just released with many defects to this
> file.
> > > > > > > > > > >
> > > > > > > > > > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
> > > > > > > jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> > > > > > > > > > > > I've noticed that LICENSE file contains entry
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > protobuf-<version>.jar whereas our binary artifacts seems to
> be pulling
> > > > > > > protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do
> need seems
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > as a
> > > > > > > blocker to me, but it would be great if someone else with more
> deeper
> > > > > > > licensing knowledge would confirm that. As long as this is
> fine, I'm
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > +1 on
> > > > > > > the release.
> > > > > > > > > > > >
> > > > > > > > > > > > Checked:
> > > > > > > > > > > > * Checksums, signatures seems fine
> > > > > > > > > > > > * "mvn clean compile test verify package" passes for
> source
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > artifact
> > > > > > > > > > > > * "mvn verify" passes for binary artifact
> > > > > > > > > > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems
> fine to me
> > > > > > > > > > > >
> > > > > > > > > > > > Notes:
> > > > > > > > > > > > * Filled FLUME-1719: Example export command in
> README do not
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > properly close the string
> > > > > > > > > > > > * Filled FLUME-1720: LICENSE file contain entry for
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > protobuf-<version>.jar, however proper artifact name is
> > > > > > > protobuf-java-<version>.jar
> > > > > > > > > > > >
> > > > > > > > > > > > Jarcec
> > > > > > > > > > > >
> > > > > > > > > > > > Links:
> > > > > > > > > > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> > > > > > > > > > > >
> > > > > > > > > > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
> > > > > Alten-Lorenz
> > > > > > > wrote:
> > > > > > > > > > > > > looks good to me, thanks.
> > > > > > > > > > > > >
> > > > > > > > > > > > > +1
> > > > > > > > > > > > >
> > > > > > > > > > > > > cheers,
> > > > > > > > > > > > > Alex
> > > > > > > > > > > > >
> > > > > > > > > > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <
> > > > > brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > > > > > >
> > > > > > > > > > > > > > +1 verified sigs, hashes, built docs, and
> verified smoke
> > > > > test
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
> > > > > > > brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > > > > > > > > > > > This is the second release for Apache Flume as
> a
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > > top-level
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > project,
> > > > > > > > > > > > > > > version 1.3.0. We are voting on release
> candidate rc3.
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > *** Please cast your vote within the next 72
> hours ***
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > The list of fixed issues:
> > > > >
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > The tarball (*.tar.gz), signature (*.asc), and
> > > > > checksums
> > > > > > > (*.md5, *.sha1)
> > > > > > > > > > > > > > > for the source and binary artifacts can be
> found at:
> > > > > > > > > > > > > > >
> http://people.apache.org/~brock/flume-1.3.0-rc3/
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > (Note: attempting to deploy the artifacts to
> Nexus
> > > > > appears
> > > > > > > to have
> > > > > > > > > > > > > > > locked me out of Apache LDAP so you'll have to
> use the
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > artifacts from
> > > > > > > > > > > > > > > the above link or via the source tag below.)
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > The tag to be voted on:
> > > > >
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > > > The KEYS file can be found here:
> > > > > > > > > > > > > > >
> https://dist.apache.org/repos/dist/release/flume/KEYS
> > > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > --
> > > > > > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > > --
> > > > > > > > > > > > > Alexander Alten-Lorenz
> > > > > > > > > > > > > http://mapredit.blogspot.com
> > > > > > > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > > > --
> > > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > --
> > > > > > > > > Alexander Alten-Lorenz
> > > > > > > > > http://mapredit.blogspot.com
> > > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > http://incubator.apache.org/mrunit/
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > >
> > > >
> > >
> > >
> > >
> >
> >
>

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Jarek Jarcec Cecho <ja...@apache.org>.
Might I humbly ask for review & commit FLUME-1719 for next RC?

Jarcec

On Sat, Nov 17, 2012 at 07:13:26PM -0800, Hari Shreedharan wrote:
> Done! Thanks Mike, Brock! 
> 
> 
> Hari 
> 
> -- 
> Hari Shreedharan
> 
> 
> On Saturday, November 17, 2012 at 7:05 PM, Brock Noland wrote:
> 
> > Ok, let's do another rc since the same lobs with different versions is probably bad. 
> > 
> > Also I think it's fine to commit to 1.3 without another rc since we are voting on a tag, not the branch. 
> > 
> > -- 
> > Brock Noland
> > Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
> > 
> > 
> > On Saturday, November 17, 2012 at 8:45 PM, Mike Percy wrote:
> > 
> > > Hari,
> > > You're correct that a -1 vote does not veto a release. A savvy user could
> > > delete one of the jars before deploying to production, but why make them do
> > > that.
> > > 
> > > Thanks for the commit.
> > > 
> > > Regards,
> > > Mike
> > > 
> > > On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)
> > > > wrote:
> > > 
> > > 
> > > 
> > > > Mike,
> > > > 
> > > > I committed your patch for flume-1.4, and trunk.
> > > > 
> > > > 
> > > > Brock: As I posted on the jira, if you want it committed to flume-1.3. the
> > > > RM should decide if another RC is required (looks like there are 3 PMC +1
> > > > already, unless someone withdraws the +1). If required, please cherry-pick
> > > > the patch to flume-1.3.0 and roll another RC.
> > > > 
> > > > 
> > > > Thanks,
> > > > Hari
> > > > 
> > > > --
> > > > Hari Shreedharan
> > > > 
> > > > 
> > > > On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
> > > > 
> > > > > When I was reviewing the libs I noticed that netty is in there twice
> > > > with 2
> > > > > different versions. I don't think we should ship like that.
> > > > > 
> > > > > So, -1 from me on this RC.
> > > > > 
> > > > > Details and a patch are here:
> > > > > https://issues.apache.org/jira/browse/FLUME-1723
> > > > > 
> > > > > Regards,
> > > > > Mike
> > > > > 
> > > > > On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > 
> > > > > > Thanks Hari, I have created FLUME-1722 for this purpose.
> > > > > > 
> > > > > > https://issues.apache.org/jira/browse/FLUME-1722
> > > > > > 
> > > > > > Brock
> > > > > > 
> > > > > > On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
> > > > > > <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)> wrote:
> > > > > > > +1.
> > > > > > > 
> > > > > > > Here is what I checked:
> > > > > > > 1. Checksums and signatures look good.
> > > > > > > 2. Verified license file, readme, change log and notice file (more on
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > this below).
> > > > > > > 3. Built, ran unit tests, ran an agent with the sample config - looks
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > good.
> > > > > > > 
> > > > > > > 
> > > > > > > Additional notes:
> > > > > > > The LICENSE file seems to list three libraries which we do not pull
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > 
> > > > 
> > > > in:
> > > > > > > 
> > > > > > > commons logging
> > > > > > > http-client
> > > > > > > http-core
> > > > > > > 
> > > > > > > 
> > > > > > > We probably should remove these from the license file too.
> > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > Thanks,
> > > > > > > Hari
> > > > > > > --
> > > > > > > Hari Shreedharan
> > > > > > > 
> > > > > > > 
> > > > > > > On Saturday, November 17, 2012 at 12:57 AM, Alexander Alten-Lorenz
> > > > wrote:
> > > > > > > 
> > > > > > > > +1
> > > > > > > > 
> > > > > > > > - Alex
> > > > > > > > 
> > > > > > > > On Nov 17, 2012, at 4:10 AM, Will McQueen <will@cloudera.com (mailto:will@cloudera.com)(mailto:
> > > > will@cloudera.com (mailto:will@cloudera.com)) (mailto:
> > > > > > will@cloudera.com (mailto:will@cloudera.com))> wrote:
> > > > > > > > 
> > > > > > > > > +1 on the release
> > > > > > > > > 
> > > > > > > > > On Nov 16, 2012, at 6:56 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > brock@cloudera.com (mailto:brock@cloudera.com))(mailto:
> > > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > > 
> > > > > > > > > > Thanks Jarek, I don't think it's a blocker but we should
> > > > definitely
> > > > > > fix it.
> > > > > > > > > > 
> > > > > > > > > > FWIW, hadoop's LICENSE.txt doesn't even list the licenses of
> > > > > > binaries
> > > > > > > > > > and Crunch was just released with many defects to this file.
> > > > > > > > > > 
> > > > > > > > > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
> > > > > > jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> > > > > > > > > > > I've noticed that LICENSE file contains entry
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > protobuf-<version>.jar whereas our binary artifacts seems to be pulling
> > > > > > protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > as a
> > > > > > blocker to me, but it would be great if someone else with more deeper
> > > > > > licensing knowledge would confirm that. As long as this is fine, I'm
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > +1 on
> > > > > > the release.
> > > > > > > > > > > 
> > > > > > > > > > > Checked:
> > > > > > > > > > > * Checksums, signatures seems fine
> > > > > > > > > > > * "mvn clean compile test verify package" passes for source
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > artifact
> > > > > > > > > > > * "mvn verify" passes for binary artifact
> > > > > > > > > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
> > > > > > > > > > > 
> > > > > > > > > > > Notes:
> > > > > > > > > > > * Filled FLUME-1719: Example export command in README do not
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > properly close the string
> > > > > > > > > > > * Filled FLUME-1720: LICENSE file contain entry for
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > protobuf-<version>.jar, however proper artifact name is
> > > > > > protobuf-java-<version>.jar
> > > > > > > > > > > 
> > > > > > > > > > > Jarcec
> > > > > > > > > > > 
> > > > > > > > > > > Links:
> > > > > > > > > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> > > > > > > > > > > 
> > > > > > > > > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
> > > > Alten-Lorenz
> > > > > > wrote:
> > > > > > > > > > > > looks good to me, thanks.
> > > > > > > > > > > > 
> > > > > > > > > > > > +1
> > > > > > > > > > > > 
> > > > > > > > > > > > cheers,
> > > > > > > > > > > > Alex
> > > > > > > > > > > > 
> > > > > > > > > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <
> > > > brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > > > > > 
> > > > > > > > > > > > > +1 verified sigs, hashes, built docs, and verified smoke
> > > > test
> > > > > > > > > > > > > 
> > > > > > > > > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
> > > > > > brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > > > > > > > > > > This is the second release for Apache Flume as a
> > > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > top-level
> > > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > project,
> > > > > > > > > > > > > > version 1.3.0. We are voting on release candidate rc3.
> > > > > > > > > > > > > > 
> > > > > > > > > > > > > > *** Please cast your vote within the next 72 hours ***
> > > > > > > > > > > > > > 
> > > > > > > > > > > > > > The list of fixed issues:
> > > > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > > 
> > > > > > > > > > > > > > The tarball (*.tar.gz), signature (*.asc), and
> > > > checksums
> > > > > > (*.md5, *.sha1)
> > > > > > > > > > > > > > for the source and binary artifacts can be found at:
> > > > > > > > > > > > > > http://people.apache.org/~brock/flume-1.3.0-rc3/
> > > > > > > > > > > > > > 
> > > > > > > > > > > > > > (Note: attempting to deploy the artifacts to Nexus
> > > > appears
> > > > > > to have
> > > > > > > > > > > > > > locked me out of Apache LDAP so you'll have to use the
> > > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > artifacts from
> > > > > > > > > > > > > > the above link or via the source tag below.)
> > > > > > > > > > > > > > 
> > > > > > > > > > > > > > The tag to be voted on:
> > > > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > > 
> > > > > > > > > > > > > > The KEYS file can be found here:
> > > > > > > > > > > > > > https://dist.apache.org/repos/dist/release/flume/KEYS
> > > > > > > > > > > > > > 
> > > > > > > > > > > > > 
> > > > > > > > > > > > > 
> > > > > > > > > > > > > 
> > > > > > > > > > > > > 
> > > > > > > > > > > > > 
> > > > > > > > > > > > > 
> > > > > > > > > > > > > 
> > > > > > > > > > > > > --
> > > > > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > --
> > > > > > > > > > > > Alexander Alten-Lorenz
> > > > > > > > > > > > http://mapredit.blogspot.com
> > > > > > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > --
> > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > --
> > > > > > > > Alexander Alten-Lorenz
> > > > > > > > http://mapredit.blogspot.com
> > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > --
> > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > http://incubator.apache.org/mrunit/
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > 
> > > 
> > 
> > 
> > 
> 
> 

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Mike Percy <mp...@apache.org>.
I think that's the right way to go. Sorry for causing trouble guys! :)

Regards,
Mike


On Sat, Nov 17, 2012 at 7:13 PM, Hari Shreedharan <hshreedharan@cloudera.com
> wrote:

> Done! Thanks Mike, Brock!
>
>
> Hari
>
> --
> Hari Shreedharan
>
>
> On Saturday, November 17, 2012 at 7:05 PM, Brock Noland wrote:
>
> > Ok, let's do another rc since the same lobs with different versions is
> probably bad.
> >
> > Also I think it's fine to commit to 1.3 without another rc since we are
> voting on a tag, not the branch.
> >
> > --
> > Brock Noland
> > Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
> >
> >
> > On Saturday, November 17, 2012 at 8:45 PM, Mike Percy wrote:
> >
> > > Hari,
> > > You're correct that a -1 vote does not veto a release. A savvy user
> could
> > > delete one of the jars before deploying to production, but why make
> them do
> > > that.
> > >
> > > Thanks for the commit.
> > >
> > > Regards,
> > > Mike
> > >
> > > On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <
> hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)
> > > > wrote:
> > >
> > >
> > >
> > > > Mike,
> > > >
> > > > I committed your patch for flume-1.4, and trunk.
> > > >
> > > >
> > > > Brock: As I posted on the jira, if you want it committed to
> flume-1.3. the
> > > > RM should decide if another RC is required (looks like there are 3
> PMC +1
> > > > already, unless someone withdraws the +1). If required, please
> cherry-pick
> > > > the patch to flume-1.3.0 and roll another RC.
> > > >
> > > >
> > > > Thanks,
> > > > Hari
> > > >
> > > > --
> > > > Hari Shreedharan
> > > >
> > > >
> > > > On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
> > > >
> > > > > When I was reviewing the libs I noticed that netty is in there
> twice
> > > > with 2
> > > > > different versions. I don't think we should ship like that.
> > > > >
> > > > > So, -1 from me on this RC.
> > > > >
> > > > > Details and a patch are here:
> > > > > https://issues.apache.org/jira/browse/FLUME-1723
> > > > >
> > > > > Regards,
> > > > > Mike
> > > > >
> > > > > On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <brock@cloudera.com(mailto:
> brock@cloudera.com)(mailto:
> > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > >
> > > > > > Thanks Hari, I have created FLUME-1722 for this purpose.
> > > > > >
> > > > > > https://issues.apache.org/jira/browse/FLUME-1722
> > > > > >
> > > > > > Brock
> > > > > >
> > > > > > On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
> > > > > > <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)>
> wrote:
> > > > > > > +1.
> > > > > > >
> > > > > > > Here is what I checked:
> > > > > > > 1. Checksums and signatures look good.
> > > > > > > 2. Verified license file, readme, change log and notice file
> (more on
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > this below).
> > > > > > > 3. Built, ran unit tests, ran an agent with the sample config
> - looks
> > > > > >
> > > > > >
> > > > > >
> > > > > > good.
> > > > > > >
> > > > > > >
> > > > > > > Additional notes:
> > > > > > > The LICENSE file seems to list three libraries which we do not
> pull
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > > >
> > > >
> > > >
> > > > in:
> > > > > > >
> > > > > > > commons logging
> > > > > > > http-client
> > > > > > > http-core
> > > > > > >
> > > > > > >
> > > > > > > We probably should remove these from the license file too.
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > Thanks,
> > > > > > > Hari
> > > > > > > --
> > > > > > > Hari Shreedharan
> > > > > > >
> > > > > > >
> > > > > > > On Saturday, November 17, 2012 at 12:57 AM, Alexander
> Alten-Lorenz
> > > > wrote:
> > > > > > >
> > > > > > > > +1
> > > > > > > >
> > > > > > > > - Alex
> > > > > > > >
> > > > > > > > On Nov 17, 2012, at 4:10 AM, Will McQueen <will@cloudera.com(mailto:
> will@cloudera.com)(mailto:
> > > > will@cloudera.com (mailto:will@cloudera.com)) (mailto:
> > > > > > will@cloudera.com (mailto:will@cloudera.com))> wrote:
> > > > > > > >
> > > > > > > > > +1 on the release
> > > > > > > > >
> > > > > > > > > On Nov 16, 2012, at 6:56 PM, Brock Noland <
> brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > brock@cloudera.com (mailto:brock@cloudera.com))(mailto:
> > > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > >
> > > > > > > > > > Thanks Jarek, I don't think it's a blocker but we should
> > > > definitely
> > > > > > fix it.
> > > > > > > > > >
> > > > > > > > > > FWIW, hadoop's LICENSE.txt doesn't even list the
> licenses of
> > > > > > binaries
> > > > > > > > > > and Crunch was just released with many defects to this
> file.
> > > > > > > > > >
> > > > > > > > > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
> > > > > > jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> > > > > > > > > > > I've noticed that LICENSE file contains entry
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > protobuf-<version>.jar whereas our binary artifacts seems to be
> pulling
> > > > > > protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need
> seems
> > > > > >
> > > > >
> > > >
> > > >
> > > > as a
> > > > > > blocker to me, but it would be great if someone else with more
> deeper
> > > > > > licensing knowledge would confirm that. As long as this is fine,
> I'm
> > > > > >
> > > > >
> > > >
> > > >
> > > > +1 on
> > > > > > the release.
> > > > > > > > > > >
> > > > > > > > > > > Checked:
> > > > > > > > > > > * Checksums, signatures seems fine
> > > > > > > > > > > * "mvn clean compile test verify package" passes for
> source
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > artifact
> > > > > > > > > > > * "mvn verify" passes for binary artifact
> > > > > > > > > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems fine
> to me
> > > > > > > > > > >
> > > > > > > > > > > Notes:
> > > > > > > > > > > * Filled FLUME-1719: Example export command in README
> do not
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > properly close the string
> > > > > > > > > > > * Filled FLUME-1720: LICENSE file contain entry for
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > protobuf-<version>.jar, however proper artifact name is
> > > > > > protobuf-java-<version>.jar
> > > > > > > > > > >
> > > > > > > > > > > Jarcec
> > > > > > > > > > >
> > > > > > > > > > > Links:
> > > > > > > > > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> > > > > > > > > > >
> > > > > > > > > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
> > > > Alten-Lorenz
> > > > > > wrote:
> > > > > > > > > > > > looks good to me, thanks.
> > > > > > > > > > > >
> > > > > > > > > > > > +1
> > > > > > > > > > > >
> > > > > > > > > > > > cheers,
> > > > > > > > > > > > Alex
> > > > > > > > > > > >
> > > > > > > > > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <
> > > > brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > > > > >
> > > > > > > > > > > > > +1 verified sigs, hashes, built docs, and verified
> smoke
> > > > test
> > > > > > > > > > > > >
> > > > > > > > > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
> > > > > > brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > > > > > > > > > > This is the second release for Apache Flume as a
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > >
> > > >
> > > >
> > > > top-level
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > project,
> > > > > > > > > > > > > > version 1.3.0. We are voting on release
> candidate rc3.
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > *** Please cast your vote within the next 72
> hours ***
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > The list of fixed issues:
> > > >
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > The tarball (*.tar.gz), signature (*.asc), and
> > > > checksums
> > > > > > (*.md5, *.sha1)
> > > > > > > > > > > > > > for the source and binary artifacts can be found
> at:
> > > > > > > > > > > > > > http://people.apache.org/~brock/flume-1.3.0-rc3/
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > (Note: attempting to deploy the artifacts to
> Nexus
> > > > appears
> > > > > > to have
> > > > > > > > > > > > > > locked me out of Apache LDAP so you'll have to
> use the
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > artifacts from
> > > > > > > > > > > > > > the above link or via the source tag below.)
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > The tag to be voted on:
> > > >
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > >
> > > > > > > > > > > > > > The KEYS file can be found here:
> > > > > > > > > > > > > >
> https://dist.apache.org/repos/dist/release/flume/KEYS
> > > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > >
> > > > > > > > > > > > > --
> > > > > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > >
> > > > > > > > > > > > --
> > > > > > > > > > > > Alexander Alten-Lorenz
> > > > > > > > > > > > http://mapredit.blogspot.com
> > > > > > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > > > > >
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > --
> > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > >
> > > > > > > > --
> > > > > > > > Alexander Alten-Lorenz
> > > > > > > > http://mapredit.blogspot.com
> > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > >
> > > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > >
> > > > > > --
> > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > http://incubator.apache.org/mrunit/
> > > > > >
> > > > >
> > > >
> > > >
> > >
> > >
> >
> >
> >
>
>
>

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Hari Shreedharan <hs...@cloudera.com>.
Done! Thanks Mike, Brock! 


Hari 

-- 
Hari Shreedharan


On Saturday, November 17, 2012 at 7:05 PM, Brock Noland wrote:

> Ok, let's do another rc since the same lobs with different versions is probably bad. 
> 
> Also I think it's fine to commit to 1.3 without another rc since we are voting on a tag, not the branch. 
> 
> -- 
> Brock Noland
> Sent with Sparrow (http://www.sparrowmailapp.com/?sig)
> 
> 
> On Saturday, November 17, 2012 at 8:45 PM, Mike Percy wrote:
> 
> > Hari,
> > You're correct that a -1 vote does not veto a release. A savvy user could
> > delete one of the jars before deploying to production, but why make them do
> > that.
> > 
> > Thanks for the commit.
> > 
> > Regards,
> > Mike
> > 
> > On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)
> > > wrote:
> > 
> > 
> > 
> > > Mike,
> > > 
> > > I committed your patch for flume-1.4, and trunk.
> > > 
> > > 
> > > Brock: As I posted on the jira, if you want it committed to flume-1.3. the
> > > RM should decide if another RC is required (looks like there are 3 PMC +1
> > > already, unless someone withdraws the +1). If required, please cherry-pick
> > > the patch to flume-1.3.0 and roll another RC.
> > > 
> > > 
> > > Thanks,
> > > Hari
> > > 
> > > --
> > > Hari Shreedharan
> > > 
> > > 
> > > On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
> > > 
> > > > When I was reviewing the libs I noticed that netty is in there twice
> > > with 2
> > > > different versions. I don't think we should ship like that.
> > > > 
> > > > So, -1 from me on this RC.
> > > > 
> > > > Details and a patch are here:
> > > > https://issues.apache.org/jira/browse/FLUME-1723
> > > > 
> > > > Regards,
> > > > Mike
> > > > 
> > > > On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > 
> > > > > Thanks Hari, I have created FLUME-1722 for this purpose.
> > > > > 
> > > > > https://issues.apache.org/jira/browse/FLUME-1722
> > > > > 
> > > > > Brock
> > > > > 
> > > > > On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
> > > > > <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)> wrote:
> > > > > > +1.
> > > > > > 
> > > > > > Here is what I checked:
> > > > > > 1. Checksums and signatures look good.
> > > > > > 2. Verified license file, readme, change log and notice file (more on
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > this below).
> > > > > > 3. Built, ran unit tests, ran an agent with the sample config - looks
> > > > > 
> > > > > 
> > > > > 
> > > > > good.
> > > > > > 
> > > > > > 
> > > > > > Additional notes:
> > > > > > The LICENSE file seems to list three libraries which we do not pull
> > > > > > 
> > > > > 
> > > > > 
> > > > 
> > > > 
> > > 
> > > 
> > > in:
> > > > > > 
> > > > > > commons logging
> > > > > > http-client
> > > > > > http-core
> > > > > > 
> > > > > > 
> > > > > > We probably should remove these from the license file too.
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > Thanks,
> > > > > > Hari
> > > > > > --
> > > > > > Hari Shreedharan
> > > > > > 
> > > > > > 
> > > > > > On Saturday, November 17, 2012 at 12:57 AM, Alexander Alten-Lorenz
> > > wrote:
> > > > > > 
> > > > > > > +1
> > > > > > > 
> > > > > > > - Alex
> > > > > > > 
> > > > > > > On Nov 17, 2012, at 4:10 AM, Will McQueen <will@cloudera.com (mailto:will@cloudera.com)(mailto:
> > > will@cloudera.com (mailto:will@cloudera.com)) (mailto:
> > > > > will@cloudera.com (mailto:will@cloudera.com))> wrote:
> > > > > > > 
> > > > > > > > +1 on the release
> > > > > > > > 
> > > > > > > > On Nov 16, 2012, at 6:56 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > brock@cloudera.com (mailto:brock@cloudera.com))(mailto:
> > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > 
> > > > > > > > > Thanks Jarek, I don't think it's a blocker but we should
> > > definitely
> > > > > fix it.
> > > > > > > > > 
> > > > > > > > > FWIW, hadoop's LICENSE.txt doesn't even list the licenses of
> > > > > binaries
> > > > > > > > > and Crunch was just released with many defects to this file.
> > > > > > > > > 
> > > > > > > > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
> > > > > jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> > > > > > > > > > I've noticed that LICENSE file contains entry
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > protobuf-<version>.jar whereas our binary artifacts seems to be pulling
> > > > > protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems
> > > > > 
> > > > 
> > > 
> > > 
> > > as a
> > > > > blocker to me, but it would be great if someone else with more deeper
> > > > > licensing knowledge would confirm that. As long as this is fine, I'm
> > > > > 
> > > > 
> > > 
> > > 
> > > +1 on
> > > > > the release.
> > > > > > > > > > 
> > > > > > > > > > Checked:
> > > > > > > > > > * Checksums, signatures seems fine
> > > > > > > > > > * "mvn clean compile test verify package" passes for source
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > artifact
> > > > > > > > > > * "mvn verify" passes for binary artifact
> > > > > > > > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
> > > > > > > > > > 
> > > > > > > > > > Notes:
> > > > > > > > > > * Filled FLUME-1719: Example export command in README do not
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > properly close the string
> > > > > > > > > > * Filled FLUME-1720: LICENSE file contain entry for
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > protobuf-<version>.jar, however proper artifact name is
> > > > > protobuf-java-<version>.jar
> > > > > > > > > > 
> > > > > > > > > > Jarcec
> > > > > > > > > > 
> > > > > > > > > > Links:
> > > > > > > > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> > > > > > > > > > 
> > > > > > > > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
> > > Alten-Lorenz
> > > > > wrote:
> > > > > > > > > > > looks good to me, thanks.
> > > > > > > > > > > 
> > > > > > > > > > > +1
> > > > > > > > > > > 
> > > > > > > > > > > cheers,
> > > > > > > > > > > Alex
> > > > > > > > > > > 
> > > > > > > > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <
> > > brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > > > > 
> > > > > > > > > > > > +1 verified sigs, hashes, built docs, and verified smoke
> > > test
> > > > > > > > > > > > 
> > > > > > > > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
> > > > > brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > > > > > > > > > This is the second release for Apache Flume as a
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > 
> > > 
> > > 
> > > top-level
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > project,
> > > > > > > > > > > > > version 1.3.0. We are voting on release candidate rc3.
> > > > > > > > > > > > > 
> > > > > > > > > > > > > *** Please cast your vote within the next 72 hours ***
> > > > > > > > > > > > > 
> > > > > > > > > > > > > The list of fixed issues:
> > > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > 
> > > > > > > > > > > > > The tarball (*.tar.gz), signature (*.asc), and
> > > checksums
> > > > > (*.md5, *.sha1)
> > > > > > > > > > > > > for the source and binary artifacts can be found at:
> > > > > > > > > > > > > http://people.apache.org/~brock/flume-1.3.0-rc3/
> > > > > > > > > > > > > 
> > > > > > > > > > > > > (Note: attempting to deploy the artifacts to Nexus
> > > appears
> > > > > to have
> > > > > > > > > > > > > locked me out of Apache LDAP so you'll have to use the
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > artifacts from
> > > > > > > > > > > > > the above link or via the source tag below.)
> > > > > > > > > > > > > 
> > > > > > > > > > > > > The tag to be voted on:
> > > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > > 
> > > > > > > > > > > > > The KEYS file can be found here:
> > > > > > > > > > > > > https://dist.apache.org/repos/dist/release/flume/KEYS
> > > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > 
> > > > > > > > > > > > --
> > > > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > --
> > > > > > > > > > > Alexander Alten-Lorenz
> > > > > > > > > > > http://mapredit.blogspot.com
> > > > > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > --
> > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > http://incubator.apache.org/mrunit/
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > 
> > > > > > > --
> > > > > > > Alexander Alten-Lorenz
> > > > > > > http://mapredit.blogspot.com
> > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > 
> > > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > 
> > > > > --
> > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > http://incubator.apache.org/mrunit/
> > > > > 
> > > > 
> > > 
> > > 
> > 
> > 
> 
> 
> 



Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Brock Noland <br...@cloudera.com>.
Ok, let's do another rc since the same lobs with different versions is probably bad.  

Also I think it's fine to commit to 1.3 without another rc since we are voting on a tag, not the branch.  

-- 
Brock Noland
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)


On Saturday, November 17, 2012 at 8:45 PM, Mike Percy wrote:

> Hari,
> You're correct that a -1 vote does not veto a release. A savvy user could
> delete one of the jars before deploying to production, but why make them do
> that.
> 
> Thanks for the commit.
> 
> Regards,
> Mike
> 
> On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <hshreedharan@cloudera.com
> > wrote:
> 
> 
> > Mike,
> > 
> > I committed your patch for flume-1.4, and trunk.
> > 
> > 
> > Brock: As I posted on the jira, if you want it committed to flume-1.3. the
> > RM should decide if another RC is required (looks like there are 3 PMC +1
> > already, unless someone withdraws the +1). If required, please cherry-pick
> > the patch to flume-1.3.0 and roll another RC.
> > 
> > 
> > Thanks,
> > Hari
> > 
> > --
> > Hari Shreedharan
> > 
> > 
> > On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
> > 
> > > When I was reviewing the libs I noticed that netty is in there twice
> > with 2
> > > different versions. I don't think we should ship like that.
> > > 
> > > So, -1 from me on this RC.
> > > 
> > > Details and a patch are here:
> > > https://issues.apache.org/jira/browse/FLUME-1723
> > > 
> > > Regards,
> > > Mike
> > > 
> > > On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <brock@cloudera.com(mailto:
> > brock@cloudera.com)> wrote:
> > > 
> > > > Thanks Hari, I have created FLUME-1722 for this purpose.
> > > > 
> > > > https://issues.apache.org/jira/browse/FLUME-1722
> > > > 
> > > > Brock
> > > > 
> > > > On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
> > > > <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)> wrote:
> > > > > +1.
> > > > > 
> > > > > Here is what I checked:
> > > > > 1. Checksums and signatures look good.
> > > > > 2. Verified license file, readme, change log and notice file (more on
> > > > > 
> > > > 
> > > > 
> > > > this below).
> > > > > 3. Built, ran unit tests, ran an agent with the sample config - looks
> > > > 
> > > > 
> > > > good.
> > > > > 
> > > > > 
> > > > > Additional notes:
> > > > > The LICENSE file seems to list three libraries which we do not pull
> > > > > 
> > > > 
> > > > 
> > > 
> > > 
> > 
> > in:
> > > > > 
> > > > > commons logging
> > > > > http-client
> > > > > http-core
> > > > > 
> > > > > 
> > > > > We probably should remove these from the license file too.
> > > > > 
> > > > > 
> > > > > 
> > > > > Thanks,
> > > > > Hari
> > > > > --
> > > > > Hari Shreedharan
> > > > > 
> > > > > 
> > > > > On Saturday, November 17, 2012 at 12:57 AM, Alexander Alten-Lorenz
> > wrote:
> > > > > 
> > > > > > +1
> > > > > > 
> > > > > > - Alex
> > > > > > 
> > > > > > On Nov 17, 2012, at 4:10 AM, Will McQueen <will@cloudera.com(mailto:
> > will@cloudera.com) (mailto:
> > > > will@cloudera.com (mailto:will@cloudera.com))> wrote:
> > > > > > 
> > > > > > > +1 on the release
> > > > > > > 
> > > > > > > On Nov 16, 2012, at 6:56 PM, Brock Noland <brock@cloudera.com(mailto:
> > brock@cloudera.com)(mailto:
> > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > 
> > > > > > > > Thanks Jarek, I don't think it's a blocker but we should
> > definitely
> > > > fix it.
> > > > > > > > 
> > > > > > > > FWIW, hadoop's LICENSE.txt doesn't even list the licenses of
> > > > binaries
> > > > > > > > and Crunch was just released with many defects to this file.
> > > > > > > > 
> > > > > > > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
> > > > jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> > > > > > > > > I've noticed that LICENSE file contains entry
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > protobuf-<version>.jar whereas our binary artifacts seems to be pulling
> > > > protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems
> > > > 
> > > 
> > 
> > as a
> > > > blocker to me, but it would be great if someone else with more deeper
> > > > licensing knowledge would confirm that. As long as this is fine, I'm
> > > > 
> > > 
> > 
> > +1 on
> > > > the release.
> > > > > > > > > 
> > > > > > > > > Checked:
> > > > > > > > > * Checksums, signatures seems fine
> > > > > > > > > * "mvn clean compile test verify package" passes for source
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > artifact
> > > > > > > > > * "mvn verify" passes for binary artifact
> > > > > > > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
> > > > > > > > > 
> > > > > > > > > Notes:
> > > > > > > > > * Filled FLUME-1719: Example export command in README do not
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > properly close the string
> > > > > > > > > * Filled FLUME-1720: LICENSE file contain entry for
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > protobuf-<version>.jar, however proper artifact name is
> > > > protobuf-java-<version>.jar
> > > > > > > > > 
> > > > > > > > > Jarcec
> > > > > > > > > 
> > > > > > > > > Links:
> > > > > > > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> > > > > > > > > 
> > > > > > > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
> > Alten-Lorenz
> > > > wrote:
> > > > > > > > > > looks good to me, thanks.
> > > > > > > > > > 
> > > > > > > > > > +1
> > > > > > > > > > 
> > > > > > > > > > cheers,
> > > > > > > > > > Alex
> > > > > > > > > > 
> > > > > > > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <
> > brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > > > 
> > > > > > > > > > > +1 verified sigs, hashes, built docs, and verified smoke
> > test
> > > > > > > > > > > 
> > > > > > > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
> > > > brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > > > > > > > > This is the second release for Apache Flume as a
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > 
> > 
> > top-level
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > project,
> > > > > > > > > > > > version 1.3.0. We are voting on release candidate rc3.
> > > > > > > > > > > > 
> > > > > > > > > > > > *** Please cast your vote within the next 72 hours ***
> > > > > > > > > > > > 
> > > > > > > > > > > > The list of fixed issues:
> > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > 
> > > > > > > > > > > > The tarball (*.tar.gz), signature (*.asc), and
> > checksums
> > > > (*.md5, *.sha1)
> > > > > > > > > > > > for the source and binary artifacts can be found at:
> > > > > > > > > > > > http://people.apache.org/~brock/flume-1.3.0-rc3/
> > > > > > > > > > > > 
> > > > > > > > > > > > (Note: attempting to deploy the artifacts to Nexus
> > appears
> > > > to have
> > > > > > > > > > > > locked me out of Apache LDAP so you'll have to use the
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > artifacts from
> > > > > > > > > > > > the above link or via the source tag below.)
> > > > > > > > > > > > 
> > > > > > > > > > > > The tag to be voted on:
> > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > > > 
> > > > > > > > > > > > The KEYS file can be found here:
> > > > > > > > > > > > https://dist.apache.org/repos/dist/release/flume/KEYS
> > > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > 
> > > > > > > > > > > --
> > > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > http://incubator.apache.org/mrunit/
> > > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > 
> > > > > > > > > > --
> > > > > > > > > > Alexander Alten-Lorenz
> > > > > > > > > > http://mapredit.blogspot.com
> > > > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > --
> > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > http://incubator.apache.org/mrunit/
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > --
> > > > > > Alexander Alten-Lorenz
> > > > > > http://mapredit.blogspot.com
> > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > 
> > > > 
> > > > 
> > > > --
> > > > Apache MRUnit - Unit testing MapReduce -
> > > > http://incubator.apache.org/mrunit/
> > > > 
> > > 
> > 
> > 
> 
> 
> 



Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Mike Percy <mp...@apache.org>.
Hari,
You're correct that a -1 vote does not veto a release. A savvy user could
delete one of the jars before deploying to production, but why make them do
that.

Thanks for the commit.

Regards,
Mike

On Sat, Nov 17, 2012 at 6:12 PM, Hari Shreedharan <hshreedharan@cloudera.com
> wrote:

> Mike,
>
> I committed your patch for flume-1.4, and trunk.
>
>
> Brock: As I posted on the jira, if you want it committed to flume-1.3. the
> RM should decide if another RC is required (looks like there are 3 PMC +1
> already, unless someone withdraws the +1). If required, please cherry-pick
> the patch to flume-1.3.0 and roll another RC.
>
>
> Thanks,
> Hari
>
> --
> Hari Shreedharan
>
>
> On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:
>
> > When I was reviewing the libs I noticed that netty is in there twice
> with 2
> > different versions. I don't think we should ship like that.
> >
> > So, -1 from me on this RC.
> >
> > Details and a patch are here:
> > https://issues.apache.org/jira/browse/FLUME-1723
> >
> > Regards,
> > Mike
> >
> > On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <brock@cloudera.com(mailto:
> brock@cloudera.com)> wrote:
> >
> > > Thanks Hari, I have created FLUME-1722 for this purpose.
> > >
> > > https://issues.apache.org/jira/browse/FLUME-1722
> > >
> > > Brock
> > >
> > > On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
> > > <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)> wrote:
> > > > +1.
> > > >
> > > > Here is what I checked:
> > > > 1. Checksums and signatures look good.
> > > > 2. Verified license file, readme, change log and notice file (more on
> > > >
> > >
> > > this below).
> > > > 3. Built, ran unit tests, ran an agent with the sample config - looks
> > >
> > > good.
> > > >
> > > >
> > > > Additional notes:
> > > > The LICENSE file seems to list three libraries which we do not pull
> in:
> > > >
> > > > commons logging
> > > > http-client
> > > > http-core
> > > >
> > > >
> > > > We probably should remove these from the license file too.
> > > >
> > > >
> > > >
> > > > Thanks,
> > > > Hari
> > > > --
> > > > Hari Shreedharan
> > > >
> > > >
> > > > On Saturday, November 17, 2012 at 12:57 AM, Alexander Alten-Lorenz
> wrote:
> > > >
> > > > > +1
> > > > >
> > > > > - Alex
> > > > >
> > > > > On Nov 17, 2012, at 4:10 AM, Will McQueen <will@cloudera.com(mailto:
> will@cloudera.com) (mailto:
> > > will@cloudera.com (mailto:will@cloudera.com))> wrote:
> > > > >
> > > > > > +1 on the release
> > > > > >
> > > > > > On Nov 16, 2012, at 6:56 PM, Brock Noland <brock@cloudera.com(mailto:
> brock@cloudera.com)(mailto:
> > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > >
> > > > > > > Thanks Jarek, I don't think it's a blocker but we should
> definitely
> > > fix it.
> > > > > > >
> > > > > > > FWIW, hadoop's LICENSE.txt doesn't even list the licenses of
> > > binaries
> > > > > > > and Crunch was just released with many defects to this file.
> > > > > > >
> > > > > > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
> > > jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> > > > > > > > I've noticed that LICENSE file contains entry
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > protobuf-<version>.jar whereas our binary artifacts seems to be pulling
> > > protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems
> as a
> > > blocker to me, but it would be great if someone else with more deeper
> > > licensing knowledge would confirm that. As long as this is fine, I'm
> +1 on
> > > the release.
> > > > > > > >
> > > > > > > > Checked:
> > > > > > > > * Checksums, signatures seems fine
> > > > > > > > * "mvn clean compile test verify package" passes for source
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > artifact
> > > > > > > > * "mvn verify" passes for binary artifact
> > > > > > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
> > > > > > > >
> > > > > > > > Notes:
> > > > > > > > * Filled FLUME-1719: Example export command in README do not
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > properly close the string
> > > > > > > > * Filled FLUME-1720: LICENSE file contain entry for
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > protobuf-<version>.jar, however proper artifact name is
> > > protobuf-java-<version>.jar
> > > > > > > >
> > > > > > > > Jarcec
> > > > > > > >
> > > > > > > > Links:
> > > > > > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> > > > > > > >
> > > > > > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander
> Alten-Lorenz
> > > wrote:
> > > > > > > > > looks good to me, thanks.
> > > > > > > > >
> > > > > > > > > +1
> > > > > > > > >
> > > > > > > > > cheers,
> > > > > > > > > Alex
> > > > > > > > >
> > > > > > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <
> brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > >
> > > > > > > > > > +1 verified sigs, hashes, built docs, and verified smoke
> test
> > > > > > > > > >
> > > > > > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
> > > brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > > > > > > > This is the second release for Apache Flume as a
> top-level
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > project,
> > > > > > > > > > > version 1.3.0. We are voting on release candidate rc3.
> > > > > > > > > > >
> > > > > > > > > > > *** Please cast your vote within the next 72 hours ***
> > > > > > > > > > >
> > > > > > > > > > > The list of fixed issues:
> > >
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > >
> > > > > > > > > > > The tarball (*.tar.gz), signature (*.asc), and
> checksums
> > > (*.md5, *.sha1)
> > > > > > > > > > > for the source and binary artifacts can be found at:
> > > > > > > > > > > http://people.apache.org/~brock/flume-1.3.0-rc3/
> > > > > > > > > > >
> > > > > > > > > > > (Note: attempting to deploy the artifacts to Nexus
> appears
> > > to have
> > > > > > > > > > > locked me out of Apache LDAP so you'll have to use the
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > artifacts from
> > > > > > > > > > > the above link or via the source tag below.)
> > > > > > > > > > >
> > > > > > > > > > > The tag to be voted on:
> > >
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > >
> > > > > > > > > > > The KEYS file can be found here:
> > > > > > > > > > > https://dist.apache.org/repos/dist/release/flume/KEYS
> > > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > >
> > > > > > > > > > --
> > > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > >
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > http://incubator.apache.org/mrunit/
> > > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > >
> > > > > > > > > --
> > > > > > > > > Alexander Alten-Lorenz
> > > > > > > > > http://mapredit.blogspot.com
> > > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > >
> > > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > >
> > > > > > > --
> > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > >
> > > > > >
> > > > >
> > > >
> > >
> > > http://incubator.apache.org/mrunit/
> > > > > > >
> > > > > >
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Alexander Alten-Lorenz
> > > > > http://mapredit.blogspot.com
> > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > >
> > > >
> > >
> > >
> > >
> > >
> > > --
> > > Apache MRUnit - Unit testing MapReduce -
> > > http://incubator.apache.org/mrunit/
> > >
> >
> >
> >
>
>
>

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Hari Shreedharan <hs...@cloudera.com>.
Mike, 

I committed your patch for flume-1.4, and trunk. 


Brock: As I posted on the jira, if you want it committed to flume-1.3. the RM should decide if another RC is required (looks like there are 3 PMC +1 already, unless someone withdraws the +1). If required, please cherry-pick the patch to flume-1.3.0 and roll another RC.


Thanks,
Hari

-- 
Hari Shreedharan


On Saturday, November 17, 2012 at 5:58 PM, Mike Percy wrote:

> When I was reviewing the libs I noticed that netty is in there twice with 2
> different versions. I don't think we should ship like that.
> 
> So, -1 from me on this RC.
> 
> Details and a patch are here:
> https://issues.apache.org/jira/browse/FLUME-1723
> 
> Regards,
> Mike
> 
> On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> 
> > Thanks Hari, I have created FLUME-1722 for this purpose.
> > 
> > https://issues.apache.org/jira/browse/FLUME-1722
> > 
> > Brock
> > 
> > On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
> > <hshreedharan@cloudera.com (mailto:hshreedharan@cloudera.com)> wrote:
> > > +1.
> > > 
> > > Here is what I checked:
> > > 1. Checksums and signatures look good.
> > > 2. Verified license file, readme, change log and notice file (more on
> > > 
> > 
> > this below).
> > > 3. Built, ran unit tests, ran an agent with the sample config - looks
> > 
> > good.
> > > 
> > > 
> > > Additional notes:
> > > The LICENSE file seems to list three libraries which we do not pull in:
> > > 
> > > commons logging
> > > http-client
> > > http-core
> > > 
> > > 
> > > We probably should remove these from the license file too.
> > > 
> > > 
> > > 
> > > Thanks,
> > > Hari
> > > --
> > > Hari Shreedharan
> > > 
> > > 
> > > On Saturday, November 17, 2012 at 12:57 AM, Alexander Alten-Lorenz wrote:
> > > 
> > > > +1
> > > > 
> > > > - Alex
> > > > 
> > > > On Nov 17, 2012, at 4:10 AM, Will McQueen <will@cloudera.com (mailto:will@cloudera.com) (mailto:
> > will@cloudera.com (mailto:will@cloudera.com))> wrote:
> > > > 
> > > > > +1 on the release
> > > > > 
> > > > > On Nov 16, 2012, at 6:56 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > 
> > > > > > Thanks Jarek, I don't think it's a blocker but we should definitely
> > fix it.
> > > > > > 
> > > > > > FWIW, hadoop's LICENSE.txt doesn't even list the licenses of
> > binaries
> > > > > > and Crunch was just released with many defects to this file.
> > > > > > 
> > > > > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
> > jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> > > > > > > I've noticed that LICENSE file contains entry
> > > > > > 
> > > > > 
> > > > 
> > > 
> > 
> > protobuf-<version>.jar whereas our binary artifacts seems to be pulling
> > protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems as a
> > blocker to me, but it would be great if someone else with more deeper
> > licensing knowledge would confirm that. As long as this is fine, I'm +1 on
> > the release.
> > > > > > > 
> > > > > > > Checked:
> > > > > > > * Checksums, signatures seems fine
> > > > > > > * "mvn clean compile test verify package" passes for source
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > 
> > 
> > artifact
> > > > > > > * "mvn verify" passes for binary artifact
> > > > > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
> > > > > > > 
> > > > > > > Notes:
> > > > > > > * Filled FLUME-1719: Example export command in README do not
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > 
> > 
> > properly close the string
> > > > > > > * Filled FLUME-1720: LICENSE file contain entry for
> > > > > > 
> > > > > 
> > > > 
> > > 
> > 
> > protobuf-<version>.jar, however proper artifact name is
> > protobuf-java-<version>.jar
> > > > > > > 
> > > > > > > Jarcec
> > > > > > > 
> > > > > > > Links:
> > > > > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> > > > > > > 
> > > > > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander Alten-Lorenz
> > wrote:
> > > > > > > > looks good to me, thanks.
> > > > > > > > 
> > > > > > > > +1
> > > > > > > > 
> > > > > > > > cheers,
> > > > > > > > Alex
> > > > > > > > 
> > > > > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)(mailto:
> > brock@cloudera.com (mailto:brock@cloudera.com))> wrote:
> > > > > > > > 
> > > > > > > > > +1 verified sigs, hashes, built docs, and verified smoke test
> > > > > > > > > 
> > > > > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
> > brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > > > > > > This is the second release for Apache Flume as a top-level
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > 
> > 
> > project,
> > > > > > > > > > version 1.3.0. We are voting on release candidate rc3.
> > > > > > > > > > 
> > > > > > > > > > *** Please cast your vote within the next 72 hours ***
> > > > > > > > > > 
> > > > > > > > > > The list of fixed issues:
> > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > 
> > > > > > > > > > The tarball (*.tar.gz), signature (*.asc), and checksums
> > (*.md5, *.sha1)
> > > > > > > > > > for the source and binary artifacts can be found at:
> > > > > > > > > > http://people.apache.org/~brock/flume-1.3.0-rc3/
> > > > > > > > > > 
> > > > > > > > > > (Note: attempting to deploy the artifacts to Nexus appears
> > to have
> > > > > > > > > > locked me out of Apache LDAP so you'll have to use the
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > 
> > 
> > artifacts from
> > > > > > > > > > the above link or via the source tag below.)
> > > > > > > > > > 
> > > > > > > > > > The tag to be voted on:
> > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> > > > > > > > > > 
> > > > > > > > > > The KEYS file can be found here:
> > > > > > > > > > https://dist.apache.org/repos/dist/release/flume/KEYS
> > > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > 
> > > > > > > > > --
> > > > > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > > > > 
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > 
> > > > 
> > > 
> > 
> > http://incubator.apache.org/mrunit/
> > > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > 
> > > > > > > > --
> > > > > > > > Alexander Alten-Lorenz
> > > > > > > > http://mapredit.blogspot.com
> > > > > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > > > > 
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > --
> > > > > > Apache MRUnit - Unit testing MapReduce -
> > > > > > 
> > > > > 
> > > > 
> > > 
> > 
> > http://incubator.apache.org/mrunit/
> > > > > > 
> > > > > 
> > > > 
> > > > 
> > > > 
> > > > --
> > > > Alexander Alten-Lorenz
> > > > http://mapredit.blogspot.com
> > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > 
> > > 
> > 
> > 
> > 
> > 
> > --
> > Apache MRUnit - Unit testing MapReduce -
> > http://incubator.apache.org/mrunit/
> > 
> 
> 
> 



Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Mike Percy <mp...@apache.org>.
When I was reviewing the libs I noticed that netty is in there twice with 2
different versions. I don't think we should ship like that.

So, -1 from me on this RC.

Details and a patch are here:
https://issues.apache.org/jira/browse/FLUME-1723

Regards,
Mike

On Sat, Nov 17, 2012 at 3:33 PM, Brock Noland <br...@cloudera.com> wrote:

> Thanks Hari, I have created FLUME-1722 for this purpose.
>
> https://issues.apache.org/jira/browse/FLUME-1722
>
> Brock
>
> On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
> <hs...@cloudera.com> wrote:
> > +1.
> >
> > Here is what I checked:
> > 1. Checksums and signatures look good.
> > 2. Verified license file, readme, change log and notice file (more on
> this below).
> > 3. Built, ran unit tests, ran an agent with the sample config - looks
> good.
> >
> >
> > Additional notes:
> > The LICENSE file seems to list three libraries which we do not pull in:
> >
> > commons logging
> > http-client
> > http-core
> >
> >
> > We probably should remove these from the license file too.
> >
> >
> >
> > Thanks,
> > Hari
> > --
> > Hari Shreedharan
> >
> >
> > On Saturday, November 17, 2012 at 12:57 AM, Alexander Alten-Lorenz wrote:
> >
> >> +1
> >>
> >> - Alex
> >>
> >> On Nov 17, 2012, at 4:10 AM, Will McQueen <will@cloudera.com (mailto:
> will@cloudera.com)> wrote:
> >>
> >> > +1 on the release
> >> >
> >> > On Nov 16, 2012, at 6:56 PM, Brock Noland <brock@cloudera.com(mailto:
> brock@cloudera.com)> wrote:
> >> >
> >> > > Thanks Jarek, I don't think it's a blocker but we should definitely
> fix it.
> >> > >
> >> > > FWIW, hadoop's LICENSE.txt doesn't even list the licenses of
> binaries
> >> > > and Crunch was just released with many defects to this file.
> >> > >
> >> > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <
> jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> >> > > > I've noticed that LICENSE file contains entry
> protobuf-<version>.jar whereas our binary artifacts seems to be pulling
> protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems as a
> blocker to me, but it would be great if someone else with more deeper
> licensing knowledge would confirm that. As long as this is fine, I'm +1 on
> the release.
> >> > > >
> >> > > > Checked:
> >> > > > * Checksums, signatures seems fine
> >> > > > * "mvn clean compile test verify package" passes for source
> artifact
> >> > > > * "mvn verify" passes for binary artifact
> >> > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
> >> > > >
> >> > > > Notes:
> >> > > > * Filled FLUME-1719: Example export command in README do not
> properly close the string
> >> > > > * Filled FLUME-1720: LICENSE file contain entry for
> protobuf-<version>.jar, however proper artifact name is
> protobuf-java-<version>.jar
> >> > > >
> >> > > > Jarcec
> >> > > >
> >> > > > Links:
> >> > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> >> > > >
> >> > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander Alten-Lorenz
> wrote:
> >> > > > > looks good to me, thanks.
> >> > > > >
> >> > > > > +1
> >> > > > >
> >> > > > > cheers,
> >> > > > > Alex
> >> > > > >
> >> > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <brock@cloudera.com(mailto:
> brock@cloudera.com)> wrote:
> >> > > > >
> >> > > > > > +1 verified sigs, hashes, built docs, and verified smoke test
> >> > > > > >
> >> > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <
> brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> >> > > > > > > This is the second release for Apache Flume as a top-level
> project,
> >> > > > > > > version 1.3.0. We are voting on release candidate rc3.
> >> > > > > > >
> >> > > > > > > *** Please cast your vote within the next 72 hours ***
> >> > > > > > >
> >> > > > > > > The list of fixed issues:
> >> > > > > > >
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> >> > > > > > >
> >> > > > > > > The tarball (*.tar.gz), signature (*.asc), and checksums
> (*.md5, *.sha1)
> >> > > > > > > for the source and binary artifacts can be found at:
> >> > > > > > > http://people.apache.org/~brock/flume-1.3.0-rc3/
> >> > > > > > >
> >> > > > > > > (Note: attempting to deploy the artifacts to Nexus appears
> to have
> >> > > > > > > locked me out of Apache LDAP so you'll have to use the
> artifacts from
> >> > > > > > > the above link or via the source tag below.)
> >> > > > > > >
> >> > > > > > > The tag to be voted on:
> >> > > > > > >
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> >> > > > > > >
> >> > > > > > > The KEYS file can be found here:
> >> > > > > > > https://dist.apache.org/repos/dist/release/flume/KEYS
> >> > > > > > >
> >> > > > > >
> >> > > > > >
> >> > > > > >
> >> > > > > >
> >> > > > > > --
> >> > > > > > Apache MRUnit - Unit testing MapReduce -
> http://incubator.apache.org/mrunit/
> >> > > > > >
> >> > > > >
> >> > > > >
> >> > > > > --
> >> > > > > Alexander Alten-Lorenz
> >> > > > > http://mapredit.blogspot.com
> >> > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> >> > > > >
> >> > > >
> >> > > >
> >> > >
> >> > >
> >> > >
> >> > >
> >> > > --
> >> > > Apache MRUnit - Unit testing MapReduce -
> http://incubator.apache.org/mrunit/
> >> > >
> >> >
> >> >
> >>
> >>
> >> --
> >> Alexander Alten-Lorenz
> >> http://mapredit.blogspot.com
> >> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> >>
> >>
> >
> >
>
>
>
> --
> Apache MRUnit - Unit testing MapReduce -
> http://incubator.apache.org/mrunit/
>

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Brock Noland <br...@cloudera.com>.
Thanks Hari, I have created FLUME-1722 for this purpose.

https://issues.apache.org/jira/browse/FLUME-1722

Brock

On Sat, Nov 17, 2012 at 5:29 PM, Hari Shreedharan
<hs...@cloudera.com> wrote:
> +1.
>
> Here is what I checked:
> 1. Checksums and signatures look good.
> 2. Verified license file, readme, change log and notice file (more on this below).
> 3. Built, ran unit tests, ran an agent with the sample config - looks good.
>
>
> Additional notes:
> The LICENSE file seems to list three libraries which we do not pull in:
>
> commons logging
> http-client
> http-core
>
>
> We probably should remove these from the license file too.
>
>
>
> Thanks,
> Hari
> --
> Hari Shreedharan
>
>
> On Saturday, November 17, 2012 at 12:57 AM, Alexander Alten-Lorenz wrote:
>
>> +1
>>
>> - Alex
>>
>> On Nov 17, 2012, at 4:10 AM, Will McQueen <will@cloudera.com (mailto:will@cloudera.com)> wrote:
>>
>> > +1 on the release
>> >
>> > On Nov 16, 2012, at 6:56 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
>> >
>> > > Thanks Jarek, I don't think it's a blocker but we should definitely fix it.
>> > >
>> > > FWIW, hadoop's LICENSE.txt doesn't even list the licenses of binaries
>> > > and Crunch was just released with many defects to this file.
>> > >
>> > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
>> > > > I've noticed that LICENSE file contains entry protobuf-<version>.jar whereas our binary artifacts seems to be pulling protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems as a blocker to me, but it would be great if someone else with more deeper licensing knowledge would confirm that. As long as this is fine, I'm +1 on the release.
>> > > >
>> > > > Checked:
>> > > > * Checksums, signatures seems fine
>> > > > * "mvn clean compile test verify package" passes for source artifact
>> > > > * "mvn verify" passes for binary artifact
>> > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
>> > > >
>> > > > Notes:
>> > > > * Filled FLUME-1719: Example export command in README do not properly close the string
>> > > > * Filled FLUME-1720: LICENSE file contain entry for protobuf-<version>.jar, however proper artifact name is protobuf-java-<version>.jar
>> > > >
>> > > > Jarcec
>> > > >
>> > > > Links:
>> > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
>> > > >
>> > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander Alten-Lorenz wrote:
>> > > > > looks good to me, thanks.
>> > > > >
>> > > > > +1
>> > > > >
>> > > > > cheers,
>> > > > > Alex
>> > > > >
>> > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
>> > > > >
>> > > > > > +1 verified sigs, hashes, built docs, and verified smoke test
>> > > > > >
>> > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
>> > > > > > > This is the second release for Apache Flume as a top-level project,
>> > > > > > > version 1.3.0. We are voting on release candidate rc3.
>> > > > > > >
>> > > > > > > *** Please cast your vote within the next 72 hours ***
>> > > > > > >
>> > > > > > > The list of fixed issues:
>> > > > > > > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
>> > > > > > >
>> > > > > > > The tarball (*.tar.gz), signature (*.asc), and checksums (*.md5, *.sha1)
>> > > > > > > for the source and binary artifacts can be found at:
>> > > > > > > http://people.apache.org/~brock/flume-1.3.0-rc3/
>> > > > > > >
>> > > > > > > (Note: attempting to deploy the artifacts to Nexus appears to have
>> > > > > > > locked me out of Apache LDAP so you'll have to use the artifacts from
>> > > > > > > the above link or via the source tag below.)
>> > > > > > >
>> > > > > > > The tag to be voted on:
>> > > > > > > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
>> > > > > > >
>> > > > > > > The KEYS file can be found here:
>> > > > > > > https://dist.apache.org/repos/dist/release/flume/KEYS
>> > > > > > >
>> > > > > >
>> > > > > >
>> > > > > >
>> > > > > >
>> > > > > > --
>> > > > > > Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
>> > > > > >
>> > > > >
>> > > > >
>> > > > > --
>> > > > > Alexander Alten-Lorenz
>> > > > > http://mapredit.blogspot.com
>> > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>> > > > >
>> > > >
>> > > >
>> > >
>> > >
>> > >
>> > >
>> > > --
>> > > Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
>> > >
>> >
>> >
>>
>>
>> --
>> Alexander Alten-Lorenz
>> http://mapredit.blogspot.com
>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>>
>>
>
>



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

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Hari Shreedharan <hs...@cloudera.com>.
+1. 

Here is what I checked:
1. Checksums and signatures look good.
2. Verified license file, readme, change log and notice file (more on this below).
3. Built, ran unit tests, ran an agent with the sample config - looks good.


Additional notes:
The LICENSE file seems to list three libraries which we do not pull in:

commons logging
http-client
http-core


We probably should remove these from the license file too. 



Thanks,
Hari
-- 
Hari Shreedharan


On Saturday, November 17, 2012 at 12:57 AM, Alexander Alten-Lorenz wrote:

> +1 
> 
> - Alex
> 
> On Nov 17, 2012, at 4:10 AM, Will McQueen <will@cloudera.com (mailto:will@cloudera.com)> wrote:
> 
> > +1 on the release
> > 
> > On Nov 16, 2012, at 6:56 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > 
> > > Thanks Jarek, I don't think it's a blocker but we should definitely fix it.
> > > 
> > > FWIW, hadoop's LICENSE.txt doesn't even list the licenses of binaries
> > > and Crunch was just released with many defects to this file.
> > > 
> > > On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <jarcec@apache.org (mailto:jarcec@apache.org)> wrote:
> > > > I've noticed that LICENSE file contains entry protobuf-<version>.jar whereas our binary artifacts seems to be pulling protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems as a blocker to me, but it would be great if someone else with more deeper licensing knowledge would confirm that. As long as this is fine, I'm +1 on the release.
> > > > 
> > > > Checked:
> > > > * Checksums, signatures seems fine
> > > > * "mvn clean compile test verify package" passes for source artifact
> > > > * "mvn verify" passes for binary artifact
> > > > * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
> > > > 
> > > > Notes:
> > > > * Filled FLUME-1719: Example export command in README do not properly close the string
> > > > * Filled FLUME-1720: LICENSE file contain entry for protobuf-<version>.jar, however proper artifact name is protobuf-java-<version>.jar
> > > > 
> > > > Jarcec
> > > > 
> > > > Links:
> > > > 1: https://issues.apache.org/jira/browse/FLUME-1720
> > > > 
> > > > On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander Alten-Lorenz wrote:
> > > > > looks good to me, thanks.
> > > > > 
> > > > > +1
> > > > > 
> > > > > cheers,
> > > > > Alex
> > > > > 
> > > > > On Nov 16, 2012, at 12:21 AM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > 
> > > > > > +1 verified sigs, hashes, built docs, and verified smoke test
> > > > > > 
> > > > > > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <brock@cloudera.com (mailto:brock@cloudera.com)> wrote:
> > > > > > > This is the second release for Apache Flume as a top-level project,
> > > > > > > version 1.3.0. We are voting on release candidate rc3.
> > > > > > > 
> > > > > > > *** Please cast your vote within the next 72 hours ***
> > > > > > > 
> > > > > > > The list of fixed issues:
> > > > > > > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> > > > > > > 
> > > > > > > The tarball (*.tar.gz), signature (*.asc), and checksums (*.md5, *.sha1)
> > > > > > > for the source and binary artifacts can be found at:
> > > > > > > http://people.apache.org/~brock/flume-1.3.0-rc3/
> > > > > > > 
> > > > > > > (Note: attempting to deploy the artifacts to Nexus appears to have
> > > > > > > locked me out of Apache LDAP so you'll have to use the artifacts from
> > > > > > > the above link or via the source tag below.)
> > > > > > > 
> > > > > > > The tag to be voted on:
> > > > > > > https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> > > > > > > 
> > > > > > > The KEYS file can be found here:
> > > > > > > https://dist.apache.org/repos/dist/release/flume/KEYS
> > > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > 
> > > > > > --
> > > > > > Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
> > > > > > 
> > > > > 
> > > > > 
> > > > > --
> > > > > Alexander Alten-Lorenz
> > > > > http://mapredit.blogspot.com
> > > > > German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> > > > > 
> > > > 
> > > > 
> > > 
> > > 
> > > 
> > > 
> > > -- 
> > > Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
> > > 
> > 
> > 
> 
> 
> --
> Alexander Alten-Lorenz
> http://mapredit.blogspot.com
> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> 
> 



Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Alexander Alten-Lorenz <wg...@gmail.com>.
+1 

- Alex

On Nov 17, 2012, at 4:10 AM, Will McQueen <wi...@cloudera.com> wrote:

> +1 on the release
> 
> On Nov 16, 2012, at 6:56 PM, Brock Noland <br...@cloudera.com> wrote:
> 
>> Thanks Jarek, I don't think it's a blocker but we should definitely fix it.
>> 
>> FWIW, hadoop's LICENSE.txt doesn't even list the licenses of binaries
>> and Crunch was just released with many defects to this file.
>> 
>> On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <ja...@apache.org> wrote:
>>> I've noticed that LICENSE file contains entry protobuf-<version>.jar whereas our binary artifacts seems to be pulling protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems as a blocker to me, but it would be great if someone else with more deeper licensing knowledge would confirm that. As long as this is fine, I'm +1 on the release.
>>> 
>>> Checked:
>>> * Checksums, signatures seems fine
>>> * "mvn clean compile test verify package" passes for source artifact
>>> * "mvn verify" passes for binary artifact
>>> * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
>>> 
>>> Notes:
>>> * Filled FLUME-1719: Example export command in README do not properly close the string
>>> * Filled FLUME-1720: LICENSE file contain entry for protobuf-<version>.jar, however proper artifact name is protobuf-java-<version>.jar
>>> 
>>> Jarcec
>>> 
>>> Links:
>>> 1: https://issues.apache.org/jira/browse/FLUME-1720
>>> 
>>> On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander Alten-Lorenz wrote:
>>>> looks good to me, thanks.
>>>> 
>>>> +1
>>>> 
>>>> cheers,
>>>> Alex
>>>> 
>>>> On Nov 16, 2012, at 12:21 AM, Brock Noland <br...@cloudera.com> wrote:
>>>> 
>>>>> +1 verified sigs, hashes, built docs, and verified smoke test
>>>>> 
>>>>> On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <br...@cloudera.com> wrote:
>>>>>> This is the second release for Apache Flume as a top-level project,
>>>>>> version 1.3.0. We are voting on release candidate rc3.
>>>>>> 
>>>>>> *** Please cast your vote within the next 72 hours ***
>>>>>> 
>>>>>> The list of fixed issues:
>>>>>> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
>>>>>> 
>>>>>> The tarball (*.tar.gz), signature (*.asc), and checksums (*.md5, *.sha1)
>>>>>> for the source and binary artifacts can be found at:
>>>>>> http://people.apache.org/~brock/flume-1.3.0-rc3/
>>>>>> 
>>>>>> (Note: attempting to deploy the artifacts to Nexus appears to have
>>>>>> locked me out of Apache LDAP so you'll have to use the artifacts from
>>>>>> the above link or via the source tag below.)
>>>>>> 
>>>>>> The tag to be voted on:
>>>>>> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
>>>>>> 
>>>>>> The KEYS file can be found here:
>>>>>> https://dist.apache.org/repos/dist/release/flume/KEYS
>>>>> 
>>>>> 
>>>>> 
>>>>> --
>>>>> Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
>>>> 
>>>> --
>>>> Alexander Alten-Lorenz
>>>> http://mapredit.blogspot.com
>>>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>> 
>> 
>> 
>> -- 
>> Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/

--
Alexander Alten-Lorenz
http://mapredit.blogspot.com
German Hadoop LinkedIn Group: http://goo.gl/N8pCF


Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Will McQueen <wi...@cloudera.com>.
+1 on the release

On Nov 16, 2012, at 6:56 PM, Brock Noland <br...@cloudera.com> wrote:

> Thanks Jarek, I don't think it's a blocker but we should definitely fix it.
> 
> FWIW, hadoop's LICENSE.txt doesn't even list the licenses of binaries
> and Crunch was just released with many defects to this file.
> 
> On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <ja...@apache.org> wrote:
>> I've noticed that LICENSE file contains entry protobuf-<version>.jar whereas our binary artifacts seems to be pulling protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems as a blocker to me, but it would be great if someone else with more deeper licensing knowledge would confirm that. As long as this is fine, I'm +1 on the release.
>> 
>> Checked:
>> * Checksums, signatures seems fine
>> * "mvn clean compile test verify package" passes for source artifact
>> * "mvn verify" passes for binary artifact
>> * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
>> 
>> Notes:
>> * Filled FLUME-1719: Example export command in README do not properly close the string
>> * Filled FLUME-1720: LICENSE file contain entry for protobuf-<version>.jar, however proper artifact name is protobuf-java-<version>.jar
>> 
>> Jarcec
>> 
>> Links:
>> 1: https://issues.apache.org/jira/browse/FLUME-1720
>> 
>> On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander Alten-Lorenz wrote:
>>> looks good to me, thanks.
>>> 
>>> +1
>>> 
>>> cheers,
>>> Alex
>>> 
>>> On Nov 16, 2012, at 12:21 AM, Brock Noland <br...@cloudera.com> wrote:
>>> 
>>>> +1 verified sigs, hashes, built docs, and verified smoke test
>>>> 
>>>> On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <br...@cloudera.com> wrote:
>>>>> This is the second release for Apache Flume as a top-level project,
>>>>> version 1.3.0. We are voting on release candidate rc3.
>>>>> 
>>>>> *** Please cast your vote within the next 72 hours ***
>>>>> 
>>>>> The list of fixed issues:
>>>>> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
>>>>> 
>>>>> The tarball (*.tar.gz), signature (*.asc), and checksums (*.md5, *.sha1)
>>>>> for the source and binary artifacts can be found at:
>>>>> http://people.apache.org/~brock/flume-1.3.0-rc3/
>>>>> 
>>>>> (Note: attempting to deploy the artifacts to Nexus appears to have
>>>>> locked me out of Apache LDAP so you'll have to use the artifacts from
>>>>> the above link or via the source tag below.)
>>>>> 
>>>>> The tag to be voted on:
>>>>> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
>>>>> 
>>>>> The KEYS file can be found here:
>>>>> https://dist.apache.org/repos/dist/release/flume/KEYS
>>>> 
>>>> 
>>>> 
>>>> --
>>>> Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
>>> 
>>> --
>>> Alexander Alten-Lorenz
>>> http://mapredit.blogspot.com
>>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> 
> 
> 
> -- 
> Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Brock Noland <br...@cloudera.com>.
Thanks Jarek, I don't think it's a blocker but we should definitely fix it.

FWIW, hadoop's LICENSE.txt doesn't even list the licenses of binaries
and Crunch was just released with many defects to this file.

On Fri, Nov 16, 2012 at 8:42 PM, Jarek Jarcec Cecho <ja...@apache.org> wrote:
> I've noticed that LICENSE file contains entry protobuf-<version>.jar whereas our binary artifacts seems to be pulling protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems as a blocker to me, but it would be great if someone else with more deeper licensing knowledge would confirm that. As long as this is fine, I'm +1 on the release.
>
> Checked:
> * Checksums, signatures seems fine
> * "mvn clean compile test verify package" passes for source artifact
> * "mvn verify" passes for binary artifact
> * CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me
>
> Notes:
> * Filled FLUME-1719: Example export command in README do not properly close the string
> * Filled FLUME-1720: LICENSE file contain entry for protobuf-<version>.jar, however proper artifact name is protobuf-java-<version>.jar
>
> Jarcec
>
> Links:
> 1: https://issues.apache.org/jira/browse/FLUME-1720
>
> On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander Alten-Lorenz wrote:
>> looks good to me, thanks.
>>
>> +1
>>
>> cheers,
>>  Alex
>>
>> On Nov 16, 2012, at 12:21 AM, Brock Noland <br...@cloudera.com> wrote:
>>
>> > +1 verified sigs, hashes, built docs, and verified smoke test
>> >
>> > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <br...@cloudera.com> wrote:
>> >> This is the second release for Apache Flume as a top-level project,
>> >> version 1.3.0. We are voting on release candidate rc3.
>> >>
>> >> *** Please cast your vote within the next 72 hours ***
>> >>
>> >> The list of fixed issues:
>> >> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
>> >>
>> >> The tarball (*.tar.gz), signature (*.asc), and checksums (*.md5, *.sha1)
>> >> for the source and binary artifacts can be found at:
>> >> http://people.apache.org/~brock/flume-1.3.0-rc3/
>> >>
>> >> (Note: attempting to deploy the artifacts to Nexus appears to have
>> >> locked me out of Apache LDAP so you'll have to use the artifacts from
>> >> the above link or via the source tag below.)
>> >>
>> >> The tag to be voted on:
>> >> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
>> >>
>> >> The KEYS file can be found here:
>> >> https://dist.apache.org/repos/dist/release/flume/KEYS
>> >
>> >
>> >
>> > --
>> > Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
>>
>> --
>> Alexander Alten-Lorenz
>> http://mapredit.blogspot.com
>> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
>>



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

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Jarek Jarcec Cecho <ja...@apache.org>.
I've noticed that LICENSE file contains entry protobuf-<version>.jar whereas our binary artifacts seems to be pulling protobuf-java-<version>.jar (filled FLUME-1720 [1]). It do need seems as a blocker to me, but it would be great if someone else with more deeper licensing knowledge would confirm that. As long as this is fine, I'm +1 on the release.

Checked:
* Checksums, signatures seems fine
* "mvn clean compile test verify package" passes for source artifact
* "mvn verify" passes for binary artifact
* CHANGELOG, NOTICE, README, DEVNOTES files seems fine to me

Notes:
* Filled FLUME-1719: Example export command in README do not properly close the string
* Filled FLUME-1720: LICENSE file contain entry for protobuf-<version>.jar, however proper artifact name is protobuf-java-<version>.jar

Jarcec

Links:
1: https://issues.apache.org/jira/browse/FLUME-1720

On Fri, Nov 16, 2012 at 07:49:13AM +0100, Alexander Alten-Lorenz wrote:
> looks good to me, thanks.
> 
> +1
> 
> cheers,
>  Alex 
> 
> On Nov 16, 2012, at 12:21 AM, Brock Noland <br...@cloudera.com> wrote:
> 
> > +1 verified sigs, hashes, built docs, and verified smoke test
> > 
> > On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <br...@cloudera.com> wrote:
> >> This is the second release for Apache Flume as a top-level project,
> >> version 1.3.0. We are voting on release candidate rc3.
> >> 
> >> *** Please cast your vote within the next 72 hours ***
> >> 
> >> The list of fixed issues:
> >> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
> >> 
> >> The tarball (*.tar.gz), signature (*.asc), and checksums (*.md5, *.sha1)
> >> for the source and binary artifacts can be found at:
> >> http://people.apache.org/~brock/flume-1.3.0-rc3/
> >> 
> >> (Note: attempting to deploy the artifacts to Nexus appears to have
> >> locked me out of Apache LDAP so you'll have to use the artifacts from
> >> the above link or via the source tag below.)
> >> 
> >> The tag to be voted on:
> >> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
> >> 
> >> The KEYS file can be found here:
> >> https://dist.apache.org/repos/dist/release/flume/KEYS
> > 
> > 
> > 
> > -- 
> > Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/
> 
> --
> Alexander Alten-Lorenz
> http://mapredit.blogspot.com
> German Hadoop LinkedIn Group: http://goo.gl/N8pCF
> 

Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Alexander Alten-Lorenz <wg...@gmail.com>.
looks good to me, thanks.

+1

cheers,
 Alex 

On Nov 16, 2012, at 12:21 AM, Brock Noland <br...@cloudera.com> wrote:

> +1 verified sigs, hashes, built docs, and verified smoke test
> 
> On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <br...@cloudera.com> wrote:
>> This is the second release for Apache Flume as a top-level project,
>> version 1.3.0. We are voting on release candidate rc3.
>> 
>> *** Please cast your vote within the next 72 hours ***
>> 
>> The list of fixed issues:
>> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
>> 
>> The tarball (*.tar.gz), signature (*.asc), and checksums (*.md5, *.sha1)
>> for the source and binary artifacts can be found at:
>> http://people.apache.org/~brock/flume-1.3.0-rc3/
>> 
>> (Note: attempting to deploy the artifacts to Nexus appears to have
>> locked me out of Apache LDAP so you'll have to use the artifacts from
>> the above link or via the source tag below.)
>> 
>> The tag to be voted on:
>> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
>> 
>> The KEYS file can be found here:
>> https://dist.apache.org/repos/dist/release/flume/KEYS
> 
> 
> 
> -- 
> Apache MRUnit - Unit testing MapReduce - http://incubator.apache.org/mrunit/

--
Alexander Alten-Lorenz
http://mapredit.blogspot.com
German Hadoop LinkedIn Group: http://goo.gl/N8pCF


Re: [VOTE] Release Apache Flume version 1.3.0-rc3

Posted by Brock Noland <br...@cloudera.com>.
+1 verified sigs, hashes, built docs, and verified smoke test

On Thu, Nov 15, 2012 at 5:16 PM, Brock Noland <br...@cloudera.com> wrote:
> This is the second release for Apache Flume as a top-level project,
> version 1.3.0. We are voting on release candidate rc3.
>
> *** Please cast your vote within the next 72 hours ***
>
> The list of fixed issues:
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=blob_plain;f=CHANGELOG;hb=656e049981df62316e0a817382087297209bf2f3
>
> The tarball (*.tar.gz), signature (*.asc), and checksums (*.md5, *.sha1)
> for the source and binary artifacts can be found at:
> http://people.apache.org/~brock/flume-1.3.0-rc3/
>
> (Note: attempting to deploy the artifacts to Nexus appears to have
> locked me out of Apache LDAP so you'll have to use the artifacts from
> the above link or via the source tag below.)
>
> The tag to be voted on:
> https://git-wip-us.apache.org/repos/asf?p=flume.git;a=commit;h=656e049981df62316e0a817382087297209bf2f3
>
> The KEYS file can be found here:
> https://dist.apache.org/repos/dist/release/flume/KEYS



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