Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Justin Mclean]

Hi,

+1 (binding)

I checked:
- incubating in name
- Signatures and hashed good
- DISCLAIMER exists
- LICENSE is fine
- NOTICE may have a minor issue (see below)
- source file have ASF headers
- No unexpected binary files
- fails to compile from source a s I have wrong version of protobuf installed???

One minor issue there are several files [1][2][3][4] that have “Copyright (c) 
2010 Yahoo!” in their headers, given the NOTICE file, I would of expected these 
files to have ASF headers OR the copyright line to not appear in NOTICE.

It would be a good idea to sign with an apache email address.

Thanks,
Justin

1. 
./benchmarks/src/main/java/org/apache/omid/benchmarks/utils/ScrambledZipfianGenerator.java
2. 
./benchmarks/src/main/java/org/apache/omid/benchmarks/utils/IntegerGenerator.java
3. ./benchmarks/src/main/java/org/apache/omid/benchmarks/utils/Generator.java
4. 
./benchmarks/src/main/java/org/apache/omid/benchmarks/utils/ZipfianGenerator.java
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[sebb]

On 24 May 2018 at 22:40, Josh Elser  wrote:
> +1 (binding), but with one issue ;)
>
> The source release looks fine, as do sigs/xsums. DISCLAIMER, LICENSE and
> NOTICE are all fine too.
>
> Only thing I see is that your new additions made in
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/KEYS must
> be reflected in
> https://dist.apache.org/repos/dist/release/incubator/omid/KEYS before a
> release happens.

Might as well maintain the keys directly in:
https://dist.apache.org/repos/dist/release/incubator/omid/KEYS

Seems no point having them in the RC directory.

>
> On 5/21/18 3:39 PM, Ohad Shacham wrote:
>>
>> Hi,
>>
>>
>> The Apache Omid community has voted on and approved a proposal
>> to release Apache Omid 0.9.0.0-incubating.
>>
>> PPMC Vote
>> Call:http://mail-archives.apache.org/mod_mbox/omid-dev/201805.mbox/%3cCAHuxFyOJZ8=ud2xuq2drbgpzznoqpvaunbxpxwyo5gs_xr+...@mail.gmail.com%3e
>>
>>
>> PPMC Vote Result:
>>
>>
>> http://mail-archives.apache.org/mod_mbox/omid-dev/201805.mbox/%3ccanqijeuua0bih+masqpirn2utrfnf7dce+ipylyja6ea03t...@mail.gmail.com%3e
>>
>>
>> PPMC Vote Summary:
>> 3 binding (IPMC member) +1 votes
>> 0 non-binding PPMC member +1 votes
>> No -1 votes
>>
>> Release notes for the 0.9.0.0
>>
>> release:https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/RELEASE_NOTES.html
>>
>>
>> Git tag for the release:
>>
>>
>> https://git-wip-us.apache.org/repos/asf/incubator-omid.git/?p=incubator-omid.git;a=tag;h=229831bc377e47431cc7cbc211803069158b5711
>>
>>
>> Sources for the release:
>>
>>
>> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/apache-omid-incubating-0.9.0.0-src.tar.gz
>>
>>
>> Source release verification: PGP Signature:
>>
>>
>> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/apache-omid-incubating-0.9.0.0-src.tar.gz.asc
>>
>>
>> SHA512 Hash:
>>
>>
>> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/apache-omid-incubating-0.9.0.0-src.tar.gz.sha512
>>
>>
>> Keys to verify the signature of the release artifact are available at:
>>
>> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/KEYS
>>
>>
>> We request the permission of IPMC to publish the above release candidate
>> as
>> Apache Omid 0.9.0.0-incubating. Please try out the package and vote. The
>> vote is open for a minimum of 72 hours or until the necessary number of
>> votes (3 binding +1s) is reached. [ ] +1 Release this package as Apache
>> Omid 0.9.0.0-incubating [ ] 0 I don't feel strongly about it, but I'm okay
>> with the release [ ] -1 Do not release this package because... Please add
>> (binding) if your vote is binding.
>>
>>
>> Thanks,
>> Ohad
>>
>
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>

-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Josh Elser]

+1 (binding), but with one issue ;)

The source release looks fine, as do sigs/xsums. DISCLAIMER, LICENSE and 
NOTICE are all fine too.


Only thing I see is that your new additions made in 
https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/KEYS 
must be reflected in 
https://dist.apache.org/repos/dist/release/incubator/omid/KEYS before a 
release happens.


On 5/21/18 3:39 PM, Ohad Shacham wrote:

Hi,


The Apache Omid community has voted on and approved a proposal
to release Apache Omid 0.9.0.0-incubating.

PPMC Vote 
Call:http://mail-archives.apache.org/mod_mbox/omid-dev/201805.mbox/%3cCAHuxFyOJZ8=ud2xuq2drbgpzznoqpvaunbxpxwyo5gs_xr+...@mail.gmail.com%3e


PPMC Vote Result:

http://mail-archives.apache.org/mod_mbox/omid-dev/201805.mbox/%3ccanqijeuua0bih+masqpirn2utrfnf7dce+ipylyja6ea03t...@mail.gmail.com%3e


PPMC Vote Summary:
3 binding (IPMC member) +1 votes
0 non-binding PPMC member +1 votes
No -1 votes

Release notes for the 0.9.0.0
release:https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/RELEASE_NOTES.html


Git tag for the release:

https://git-wip-us.apache.org/repos/asf/incubator-omid.git/?p=incubator-omid.git;a=tag;h=229831bc377e47431cc7cbc211803069158b5711


Sources for the release:

https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/apache-omid-incubating-0.9.0.0-src.tar.gz


Source release verification: PGP Signature:

https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/apache-omid-incubating-0.9.0.0-src.tar.gz.asc


SHA512 Hash:

https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/apache-omid-incubating-0.9.0.0-src.tar.gz.sha512


Keys to verify the signature of the release artifact are available at:

https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/KEYS


We request the permission of IPMC to publish the above release candidate as
Apache Omid 0.9.0.0-incubating. Please try out the package and vote. The
vote is open for a minimum of 72 hours or until the necessary number of
votes (3 binding +1s) is reached. [ ] +1 Release this package as Apache
Omid 0.9.0.0-incubating [ ] 0 I don't feel strongly about it, but I'm okay
with the release [ ] -1 Do not release this package because... Please add
(binding) if your vote is binding.


Thanks,
Ohad



-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[James Taylor]

+1 (transferring my vote over from dev list vote)

On Mon, May 21, 2018 at 12:39 PM, Ohad Shacham 
wrote:

> Hi,
>
>
> The Apache Omid community has voted on and approved a proposal
> to release Apache Omid 0.9.0.0-incubating.
>
> PPMC Vote Call:http://mail-archives.apache.org/mod_mbox/omid-dev/
> 201805.mbox/%3cCAHuxFyOJZ8=Ud2xUQ2DRbGPZZnoQpVAUnBxPXwYO5
> gs_xr+...@mail.gmail.com%3e
>
>
> PPMC Vote Result:
>
> http://mail-archives.apache.org/mod_mbox/omid-dev/201805.
> mbox/%3cCANQiJeUua0BiH+MAsqPirN2UTRFnF7dcE+iPyLYjA6eA03txLg@mail.gmail.
> com%3e
>
>
> PPMC Vote Summary:
> 3 binding (IPMC member) +1 votes
> 0 non-binding PPMC member +1 votes
> No -1 votes
>
> Release notes for the 0.9.0.0
> release:https://dist.apache.org/repos/dist/dev/incubator/
> omid/0.9.0.0-rc3/RELEASE_NOTES.html
>
>
> Git tag for the release:
>
> https://git-wip-us.apache.org/repos/asf/incubator-omid.git/?
> p=incubator-omid.git;a=tag;h=229831bc377e47431cc7cbc211803069158b5711
>
>
> Sources for the release:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc3/apache-omid-incubating-0.9.0.0-src.tar.gz
>
>
> Source release verification: PGP Signature:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc3/apache-omid-incubating-0.9.0.0-src.tar.gz.asc
>
>
> SHA512 Hash:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc3/apache-omid-incubating-0.9.0.0-src.tar.gz.sha512
>
>
> Keys to verify the signature of the release artifact are available at:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc3/KEYS
>
>
> We request the permission of IPMC to publish the above release candidate as
> Apache Omid 0.9.0.0-incubating. Please try out the package and vote. The
> vote is open for a minimum of 72 hours or until the necessary number of
> votes (3 binding +1s) is reached. [ ] +1 Release this package as Apache
> Omid 0.9.0.0-incubating [ ] 0 I don't feel strongly about it, but I'm okay
> with the release [ ] -1 Do not release this package because... Please add
> (binding) if your vote is binding.
>
>
> Thanks,
> Ohad
>

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Justin Mclean]

Hi,

> Could you please review the notice file in OMID-44
>  before I'll cook a new
> release candidate?


I’ve taken a look at NOTICE file now looks good. However where did those yahoo 
files come from? If it was from here [3] then you need to add this:
This product includes software developed by
Yahoo! Inc. (www.yahoo.com)
Copyright (c) 2010 Yahoo! Inc.  All rights reserved.

to the notice file as that is in the projects NOTICE file. [4]

You may also want to add the license for [1] (and other files) to LICENSE. It’s 
not required but a good idea to do so. [2]

Thanks,
Jutsin

1. 
https://github.com/apache/incubator-omid/blob/0c371361781957c96b20295290a167f7be3b33e2/benchmarks/src/main/java/org/apache/omid/benchmarks/utils/Generator.java
 
2. http://www.apache.org/dev/licensing-howto.html#alv2-dep
3. https://github.com/mdhim/ycsb
4. https://github.com/mdhim/ycsb/blob/master/NOTICE.txt
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Ohad Shacham]

Thanks Justin.

Could you please review the notice file in OMID-44
 before I'll cook a new
release candidate?

Thanks,
Ohad


On Thu, May 10, 2018 at 2:13 AM, Justin Mclean 
wrote:

> Hi,
>
> > Has the project taken YCSB notice file into account [1] and can you
> confirm YCSB has taken MTBT notice file into account [2]. From a casual
> look (and I could be mistaken) this doesn’t seem to be the case.
>
> BTW I don’t think this is a major issue and it can be fine in a later
> release if needed.
>
> Thanks,
> Justin
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Justin Mclean]

Hi,

> Has the project taken YCSB notice file into account [1] and can you confirm 
> YCSB has taken MTBT notice file into account [2]. From a casual look (and I 
> could be mistaken) this doesn’t seem to be the case.

BTW I don’t think this is a major issue and it can be fine in a later release 
if needed.

Thanks,
Justin
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Justin Mclean]

Hi,

> Both MTBT and Omid took files from YCSB, therefore, I don’t understand why
> we should take MTBT’s NOTICE file into account?

Has the project taken YCSB notice file into account [1] and can you confirm 
YCSB has taken MTBT notice file into account [2]. From a casual look (and I 
could be mistaken) this doesn’t seem to be the case.

Thanks,
Justin

1. https://github.com/brianfrankcooper/YCSB/blob/master/NOTICE.txt
2. https://github.com/linkedin/MTBT/blob/master/NOTICE
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Ohad Shacham]

 Hi,



Both MTBT and Omid took files from YCSB, therefore, I don’t understand why
we should take MTBT’s NOTICE file into account?



Thx,

Ohad



On Wed, May 9, 2018 at 5:14 AM, Justin Mclean  wrote:

> Hi,
>
> > @Justin - thanks for taking the time to review. Really appreciate it.
> Looks
> > like there's some confusion, though. Prior to starting this vote, you
> > approved the change on the JIRA here[1], but now you've -1'ed the
> release.
>
> AFAICS this is still unanswered:
>
> Not 100% certain but I think that the NOTICE file [5] needs to be taken
> into account, however it may be that the version of the files you have were
> taken when no NOTICE files existed? (It was committed in 2014? and the
> copyright is 2010 on those files.)
>
> Thanks,
> Justin
>
> 5. https://github.com/linkedin/MTBT/blob/master/NOTICE <
> https://github.com/linkedin/MTBT/blob/master/NOTICE>

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Justin Mclean]

Hi,

> @Justin - thanks for taking the time to review. Really appreciate it. Looks
> like there's some confusion, though. Prior to starting this vote, you
> approved the change on the JIRA here[1], but now you've -1'ed the release.

AFAICS this is still unanswered:

Not 100% certain but I think that the NOTICE file [5] needs to be taken into 
account, however it may be that the version of the files you have were taken 
when no NOTICE files existed? (It was committed in 2014? and the copyright is 
2010 on those files.)

Thanks,
Justin

5. https://github.com/linkedin/MTBT/blob/master/NOTICE 

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[James Taylor]

@Justin - thanks for taking the time to review. Really appreciate it. Looks
like there's some confusion, though. Prior to starting this vote, you
approved the change on the JIRA here[1], but now you've -1'ed the release.
At this point, we're trying to get clarification on OMID-44 so that when we
go through another vote, we make sure we've done it correctly.

Would you have time to please review the change on the JIRA so we can spin
up the next RC?

Thanks,
James


[1]
https://issues.apache.org/jira/browse/OMID-44?focusedCommentId=16424626=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16424626

On Fri, Apr 13, 2018 at 8:47 AM, Matt Sicker  wrote:

> The protobuf issue I was able to fix locally by installing protobuf@2.6
> (homebrew) and modifying my PATH for the build command. I still had failing
> tests, though.
>
> On 12 April 2018 at 19:23, Justin Mclean  wrote:
>
> > Hi,
> >
> > -1 binding as NOTICE is incorrect. The NOTICE file need to be keep as
> > small as possible [1]
> >
> > I checked:
> > - incubating in file name
> > - signatures good although it would be best to sign with an apache.org
> > email address
> > - LICENSE is fine
> > - NOTICE file contains wrong year (2016) and incorrectly lists files that
> > have 3rd party ALv2 headers. Where did these files come from? It looks
> like
> > here [4] which has a NOTICE file [4]
> > - a couple of files are missing ASF headers [2][3]
> > - no unexpected binary files
> > - Can't compile from source. Looks like I have a newer version of
> > protobuf. May be an issue for other people as well?
> >
> > Not 100% certain but I think that the NOTICE file [5] needs to be taken
> > into account, however it may be that the version of the files you have
> were
> > taken when no NOTICE files existed? (It was committed in 2014? and the
> > copyright is 2010 on those files.)
> >
> > Thanks,
> > Justin
> >
> > 1. http://www.apache.org/dev/licensing-howto.html#mod-notice
> > 2. apache-omid-incubating-0.9.0.0-src/tso-server/src/main/
> > resources/default-omid-server-configuration.yml
> > 3. apache-omid-incubating-0.9.0.0-src/benchmarks/src/main/
> > resources/default-tso-server-benchmark-config.yml
> > 4.https://github.com/linkedin/MTBT/
> > 5. https://github.com/linkedin/MTBT/blob/master/NOTICE
> > -
> > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > For additional commands, e-mail: general-h...@incubator.apache.org
> >
> >
>
>
> --
> Matt Sicker 
>

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Matt Sicker]

The protobuf issue I was able to fix locally by installing protobuf@2.6
(homebrew) and modifying my PATH for the build command. I still had failing
tests, though.

On 12 April 2018 at 19:23, Justin Mclean  wrote:

> Hi,
>
> -1 binding as NOTICE is incorrect. The NOTICE file need to be keep as
> small as possible [1]
>
> I checked:
> - incubating in file name
> - signatures good although it would be best to sign with an apache.org
> email address
> - LICENSE is fine
> - NOTICE file contains wrong year (2016) and incorrectly lists files that
> have 3rd party ALv2 headers. Where did these files come from? It looks like
> here [4] which has a NOTICE file [4]
> - a couple of files are missing ASF headers [2][3]
> - no unexpected binary files
> - Can't compile from source. Looks like I have a newer version of
> protobuf. May be an issue for other people as well?
>
> Not 100% certain but I think that the NOTICE file [5] needs to be taken
> into account, however it may be that the version of the files you have were
> taken when no NOTICE files existed? (It was committed in 2014? and the
> copyright is 2010 on those files.)
>
> Thanks,
> Justin
>
> 1. http://www.apache.org/dev/licensing-howto.html#mod-notice
> 2. apache-omid-incubating-0.9.0.0-src/tso-server/src/main/
> resources/default-omid-server-configuration.yml
> 3. apache-omid-incubating-0.9.0.0-src/benchmarks/src/main/
> resources/default-tso-server-benchmark-config.yml
> 4.https://github.com/linkedin/MTBT/
> 5. https://github.com/linkedin/MTBT/blob/master/NOTICE
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>


-- 
Matt Sicker 

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Justin Mclean]

Hi,

-1 binding as NOTICE is incorrect. The NOTICE file need to be keep as small as 
possible [1]

I checked:
- incubating in file name
- signatures good although it would be best to sign with an apache.org email 
address
- LICENSE is fine
- NOTICE file contains wrong year (2016) and incorrectly lists files that have 
3rd party ALv2 headers. Where did these files come from? It looks like here [4] 
which has a NOTICE file [4]
- a couple of files are missing ASF headers [2][3]
- no unexpected binary files
- Can't compile from source. Looks like I have a newer version of protobuf. May 
be an issue for other people as well?

Not 100% certain but I think that the NOTICE file [5] needs to be taken into 
account, however it may be that the version of the files you have were taken 
when no NOTICE files existed? (It was committed in 2014? and the copyright is 
2010 on those files.)

Thanks,
Justin

1. http://www.apache.org/dev/licensing-howto.html#mod-notice
2. 
apache-omid-incubating-0.9.0.0-src/tso-server/src/main/resources/default-omid-server-configuration.yml
3. 
apache-omid-incubating-0.9.0.0-src/benchmarks/src/main/resources/default-tso-server-benchmark-config.yml
4.https://github.com/linkedin/MTBT/
5. https://github.com/linkedin/MTBT/blob/master/NOTICE
-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Alan Gates]

Forwarding my +1 from the dev list.

Alan.

On Thu, Apr 12, 2018 at 9:52 AM, Matt Sicker  wrote:

> * Signatures ok
> * Disclaimer, license, notice ok
> * Rat check ok
>
> I am, however, getting a test failure:
>
> Failed tests:
>
> TestTSOClientConnectionToTSO.testSuccessOfTSOClientReconnec
> tionsToARestartedTSOWithZKPublishing:214
> » Execution
>   TestTSOClientConnectionToTSO.testSuccessfulConnectionToTSOThroughZK:176
> »
> Execution
>
> More details:
>
> Tests run: 4, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 8.343 sec
> <<< FAILURE! - in org.apache.omid.tso.client.TestTSOClientConnectionToTSO
> testSuccessOfTSOClientReconnectionsToARestartedTSOWithZKPubl
> ishing(org.apache.omid.tso.client.TestTSOClientConnectionToTSO)
> Time elapsed: 1.738 sec  <<< FAILURE!
> java.util.concurrent.ExecutionException: java.net.NoRouteToHostException:
> No route to host
> at
> org.apache.omid.tso.client.TestTSOClientConnectionToTSO.
> testSuccessOfTSOClientReconnectionsToARestartedTSOWithZKPublishing(
> TestTSOClientConnectionToTSO.java:214)
> Caused by: java.net.NoRouteToHostException: No route to host
>
> testSuccessfulConnectionToTSOThroughZK(org.apache.omid.tso.client.
> TestTSOClientConnectionToTSO)
> Time elapsed: 1.247 sec  <<< FAILURE!
> java.util.concurrent.ExecutionException: java.net.NoRouteToHostException:
> No route to host
> at
> org.apache.omid.tso.client.TestTSOClientConnectionToTSO.
> testSuccessfulConnectionToTSOThroughZK(TestTSOClientConnectionToTSO.
> java:176)
> Caused by: java.net.NoRouteToHostException: No route to host
>
>
> On 12 April 2018 at 05:54, Ohad Shacham  wrote:
>
> > Hi,
> >
> >
> > The Apache Omid community has voted on and approved a proposal
> > to release Apache Omid 0.9.0.0-incubating.
> >
> > PPMC Vote Call:http://mail-archives.apache.org/mod_mbox/incubator-
> > omid-dev/201804.mbox/%3cCAHuxFyN_uzAb12=AS-WyVXiwn1KTwmk2SY4=
> > b1f6zktwebu...@mail.gmail.com%3e
> >
> >
> > PPMC Vote Result:
> >
> > http://mail-archives.apache.org/mod_mbox/incubator-omid-
> dev/201804.mbox/%
> > 3cCAChRgeEQg7E-V6abZaWLaxiEdmULTtcmS_w=4v2d7oye0yf...@mail.gmail.com%3e
> >
> > http://mail-archives.apache.org/mod_mbox/incubator-omid-
> dev/201804.mbox/%
> > 3cCAAF1JdhkMXvObfMD6xV=RPXhk9fg27fHR-kR6tAt=w1dweo...@mail.gmail.com%3e
> >
> >
> > PPMC Vote Summary:
> > 4 binding (IPMC member) +1 votes
> > 0 non-binding PPMC member +1 votes
> > No -1 votes
> >
> > Release notes for the 0.9.0.0
> > release:https://dist.apache.org/repos/dist/dev/incubator/
> > omid/0.9.0.0-rc2/RELEASE_NOTES.html
> >
> >
> > Git tag for the release:
> >
> > https://git-wip-us.apache.org/repos/asf/incubator-omid.git/?
> > p=incubator-omid.git;a=tag;h=264a30e825bd0a75c435888d3cc12a35b19b4405
> >
> >
> > Sources for the release:
> >
> > https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> > 0-rc2/apache-omid-incubating-0.9.0.0-src.tar.gz
> >
> >
> > Source release verification: PGP Signature:
> >
> > https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> > 0-rc2/apache-omid-incubating-0.9.0.0-src.tar.gz.asc
> >
> >
> > SHA512 Hash:
> >
> > https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> > 0-rc2/apache-omid-incubating-0.9.0.0-src.tar.gz.sha512
> >
> >
> > Keys to verify the signature of the release artifact are available at:
> >
> > https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc2/KEYS
> >
> >
> > We request the permission of IPMC to publish the above release candidate
> as
> > Apache Omid 0.9.0.0-incubating. Please try out the package and vote. The
> > vote is open for a minimum of 72 hours or until the necessary number of
> > votes (3 binding +1s) is reached. [ ] +1 Release this package as Apache
> > Omid 0.9.0.0-incubating [ ] 0 I don't feel strongly about it, but I'm
> okay
> > with the release [ ] -1 Do not release this package because... Please add
> > (binding) if your vote is binding.
> >
> >
> > Thanks,
> > Ohad
> >
>
>
>
> --
> Matt Sicker 
>

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Matt Sicker]

* Signatures ok
* Disclaimer, license, notice ok
* Rat check ok

I am, however, getting a test failure:

Failed tests:

TestTSOClientConnectionToTSO.testSuccessOfTSOClientReconnectionsToARestartedTSOWithZKPublishing:214
» Execution
  TestTSOClientConnectionToTSO.testSuccessfulConnectionToTSOThroughZK:176 »
Execution

More details:

Tests run: 4, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 8.343 sec
<<< FAILURE! - in org.apache.omid.tso.client.TestTSOClientConnectionToTSO
testSuccessOfTSOClientReconnectionsToARestartedTSOWithZKPublishing(org.apache.omid.tso.client.TestTSOClientConnectionToTSO)
Time elapsed: 1.738 sec  <<< FAILURE!
java.util.concurrent.ExecutionException: java.net.NoRouteToHostException:
No route to host
at
org.apache.omid.tso.client.TestTSOClientConnectionToTSO.testSuccessOfTSOClientReconnectionsToARestartedTSOWithZKPublishing(TestTSOClientConnectionToTSO.java:214)
Caused by: java.net.NoRouteToHostException: No route to host

testSuccessfulConnectionToTSOThroughZK(org.apache.omid.tso.client.TestTSOClientConnectionToTSO)
Time elapsed: 1.247 sec  <<< FAILURE!
java.util.concurrent.ExecutionException: java.net.NoRouteToHostException:
No route to host
at
org.apache.omid.tso.client.TestTSOClientConnectionToTSO.testSuccessfulConnectionToTSOThroughZK(TestTSOClientConnectionToTSO.java:176)
Caused by: java.net.NoRouteToHostException: No route to host


On 12 April 2018 at 05:54, Ohad Shacham  wrote:

> Hi,
>
>
> The Apache Omid community has voted on and approved a proposal
> to release Apache Omid 0.9.0.0-incubating.
>
> PPMC Vote Call:http://mail-archives.apache.org/mod_mbox/incubator-
> omid-dev/201804.mbox/%3cCAHuxFyN_uzAb12=AS-WyVXiwn1KTwmk2SY4=
> b1f6zktwebu...@mail.gmail.com%3e
>
>
> PPMC Vote Result:
>
> http://mail-archives.apache.org/mod_mbox/incubator-omid-dev/201804.mbox/%
> 3cCAChRgeEQg7E-V6abZaWLaxiEdmULTtcmS_w=4v2d7oye0yf...@mail.gmail.com%3e
>
> http://mail-archives.apache.org/mod_mbox/incubator-omid-dev/201804.mbox/%
> 3cCAAF1JdhkMXvObfMD6xV=RPXhk9fg27fHR-kR6tAt=w1dweo...@mail.gmail.com%3e
>
>
> PPMC Vote Summary:
> 4 binding (IPMC member) +1 votes
> 0 non-binding PPMC member +1 votes
> No -1 votes
>
> Release notes for the 0.9.0.0
> release:https://dist.apache.org/repos/dist/dev/incubator/
> omid/0.9.0.0-rc2/RELEASE_NOTES.html
>
>
> Git tag for the release:
>
> https://git-wip-us.apache.org/repos/asf/incubator-omid.git/?
> p=incubator-omid.git;a=tag;h=264a30e825bd0a75c435888d3cc12a35b19b4405
>
>
> Sources for the release:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc2/apache-omid-incubating-0.9.0.0-src.tar.gz
>
>
> Source release verification: PGP Signature:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc2/apache-omid-incubating-0.9.0.0-src.tar.gz.asc
>
>
> SHA512 Hash:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc2/apache-omid-incubating-0.9.0.0-src.tar.gz.sha512
>
>
> Keys to verify the signature of the release artifact are available at:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc2/KEYS
>
>
> We request the permission of IPMC to publish the above release candidate as
> Apache Omid 0.9.0.0-incubating. Please try out the package and vote. The
> vote is open for a minimum of 72 hours or until the necessary number of
> votes (3 binding +1s) is reached. [ ] +1 Release this package as Apache
> Omid 0.9.0.0-incubating [ ] 0 I don't feel strongly about it, but I'm okay
> with the release [ ] -1 Do not release this package because... Please add
> (binding) if your vote is binding.
>
>
> Thanks,
> Ohad
>



-- 
Matt Sicker 

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Justin Mclean]

Hi,

-1 binding as NOTICE is incorrect. This normally would be is a minor thing but 
you have been previously asked to fix this. [1]

Also unable to build from source please include instruction on how to build 
from source in the releasee.

I checked:
- incubating in name
- signatures and hashed good
- DISCLAIMER exists
- LICENSE is correct
- NOTICE has incorrect year 2016 and is missing information [1]
- No binary files in release
- Source files have ASF headers
- unable to build from source

When trying to build from source I get an error. It's likely that the version 
of protobuf I use is more recent than what is required. Perhaps you should 
consider updating?

Thanks,
Justin

1. 
https://lists.apache.org/thread.html/10fd534d9ea4b0a4574637ab49e5d823a47a864ec27f317f90b14196@%3Cgeneral.incubator.apache.org%3E

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Matt Sicker]

* Rat check ok
* Signatures ok

I'm having issues building from source. I fixed on issue involving protoc
by installing version 2.5 (as used in the travis config) and adding it to
my PATH for the build. However, I came across another error:

[ERROR] Failed to execute goal org.apache.maven.plugins:
maven-compiler-plugin:3.5:compile (default-compile) on project
omid-hbase-tools: Compilation failure: Compilation failure:
[ERROR] /Users/matt/Downloads/apache-omid-incubating-0.9.0.0-src/
hbase-tools/src/main/java/org/apache/omid/tools/hbase/OmidTableManager.java:[33,23]
cannot find symbol
[ERROR]   symbol:   class HBaseShims
[ERROR]   location: package org.apache.omid
[ERROR] /Users/matt/Downloads/apache-omid-incubating-0.9.0.0-src/
hbase-tools/src/main/java/org/apache/omid/tools/hbase/OmidTableManager.java:[153,13]
cannot find symbol
[ERROR]   symbol:   variable HBaseShims
[ERROR]   location: class org.apache.omid.tools.hbase.OmidTableManager

I attempted to rerun the tests with the "hbase-0" profile activated, but
that caused test failures:

Failed tests:

TestTSOClientConnectionToTSO.testSuccessOfTSOClientReconnectionsToARestartedTSOWithZKPublishing:214
» Execution
  TestTSOClientConnectionToTSO.testSuccessfulConnectionToTSOThroughZK:176 »
Execution


On 23 March 2018 at 10:33, James Taylor  wrote:

> Forwarding my +1 from the dev list.
>
> James
>
> On Thu, Mar 22, 2018 at 8:31 AM Henk P. Penning  wrote:
>
> > On Thu, 22 Mar 2018, Ohad Shacham wrote:
> >
> > > Date: Thu, 22 Mar 2018 13:29:55 +0100
> > > From: Ohad Shacham 
> > > To: general@incubator.apache.org
> > > Subject: [VOTE] Release Apache Omid 0.9.0.0 (incubating)
> > >
> > > Hi,
> > >
> > >
> > > The Apache Omid community has voted on and approved a proposal
> > > to release Apache Omid 0.9.0.0-incubating.
> >
> > > MD5 Hash:
> > >
> > >
> > https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc1/apache-omid-incubating-0.9.0.0-src.tar.gz.md5
> >
> >Please do NOT publish .md5's anymore.
> >Please DO publish a .sha1, .sha256 or .sha512 instead.
> >
> >https://www.apache.org/dev/release-distribution#sigs-and-sums
> >
> >There is no need to re-vote ; just fix dist/dev/incubator/omid/
> >before you publish.
> >
> > > Ohad
> >
> >Regards,
> >
> >Henk Penning
> >
> >    _
> > Henk P. Penning, ICT-beta R Uithof MG-403_/ \_
> > Faculty of Science, Utrecht UniversityT +31 30 253 4106 / \_/ \
> > Leuvenlaan 4, 3584CE Utrecht, NL  F +31 30 253 4553 \_/ \_/
> > http://www.staff.science.uu.nl/~penni101/ M penn...@uu.nl \_/
> >
> > -
> > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > For additional commands, e-mail: general-h...@incubator.apache.org
> >
> >
>



-- 
Matt Sicker 

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[James Taylor]

Forwarding my +1 from the dev list.

James

On Thu, Mar 22, 2018 at 8:31 AM Henk P. Penning  wrote:

> On Thu, 22 Mar 2018, Ohad Shacham wrote:
>
> > Date: Thu, 22 Mar 2018 13:29:55 +0100
> > From: Ohad Shacham 
> > To: general@incubator.apache.org
> > Subject: [VOTE] Release Apache Omid 0.9.0.0 (incubating)
> >
> > Hi,
> >
> >
> > The Apache Omid community has voted on and approved a proposal
> > to release Apache Omid 0.9.0.0-incubating.
>
> > MD5 Hash:
> >
> >
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc1/apache-omid-incubating-0.9.0.0-src.tar.gz.md5
>
>Please do NOT publish .md5's anymore.
>Please DO publish a .sha1, .sha256 or .sha512 instead.
>
>https://www.apache.org/dev/release-distribution#sigs-and-sums
>
>There is no need to re-vote ; just fix dist/dev/incubator/omid/
>before you publish.
>
> > Ohad
>
>Regards,
>
>Henk Penning
>
>    _
> Henk P. Penning, ICT-beta R Uithof MG-403_/ \_
> Faculty of Science, Utrecht UniversityT +31 30 253 4106 / \_/ \
> Leuvenlaan 4, 3584CE Utrecht, NL  F +31 30 253 4553 \_/ \_/
> http://www.staff.science.uu.nl/~penni101/ M penn...@uu.nl \_/
>
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Henk P. Penning]

On Thu, 22 Mar 2018, Ohad Shacham wrote:


Date: Thu, 22 Mar 2018 13:29:55 +0100
From: Ohad Shacham 
To: general@incubator.apache.org
Subject: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

Hi,


The Apache Omid community has voted on and approved a proposal
to release Apache Omid 0.9.0.0-incubating.



MD5 Hash:

https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc1/apache-omid-incubating-0.9.0.0-src.tar.gz.md5


  Please do NOT publish .md5's anymore.
  Please DO publish a .sha1, .sha256 or .sha512 instead.

  https://www.apache.org/dev/release-distribution#sigs-and-sums

  There is no need to re-vote ; just fix dist/dev/incubator/omid/
  before you publish.


Ohad


  Regards,

  Henk Penning

   _
Henk P. Penning, ICT-beta R Uithof MG-403_/ \_
Faculty of Science, Utrecht UniversityT +31 30 253 4106 / \_/ \
Leuvenlaan 4, 3584CE Utrecht, NL  F +31 30 253 4553 \_/ \_/
http://www.staff.science.uu.nl/~penni101/ M penn...@uu.nl \_/

-
To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
For additional commands, e-mail: general-h...@incubator.apache.org

Re: [VOTE] Release Apache Omid 0.9.0.0 (incubating)

[Alan Gates]

Forwarding my +1 from the dev list.

Alan.

On Thu, Mar 22, 2018 at 5:29 AM, Ohad Shacham 
wrote:

> Hi,
>
>
> The Apache Omid community has voted on and approved a proposal
> to release Apache Omid 0.9.0.0-incubating.
>
> PPMC Vote Call:http://mail-archives.apache.org/mod_mbox/incubator-
> omid-dev/201803.mbox/%3c1827900384.321006.1520932655...@mail.yahoo.com%3e
>
>
> PPMC Vote Result:
>
> http://mail-archives.apache.org/mod_mbox/incubator-omid-dev/201803.mbox/%
> 3cCANQiJeVst7a9oSMKx9-m0=y6ypz9dnhxxytq6p5wegjnhwj...@mail.gmail.com%3e
>
>
> PPMC Vote Summary:
> 3 binding (IPMC member) +1 votes
> 1 non-binding PPMC member +1 votes
> No -1 votes
>
> Git tag for the release:
>
> https://git-wip-us.apache.org/repos/asf/incubator-omid.git/?
> p=incubator-omid.git;a=tag;h=5dd5f86822beb43d08e91d00dd2427b4f130d6c2
>
>
> Sources for the release:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc1/apache-omid-incubating-0.9.0.0-src.tar.gz
>
>
> Source release verification: PGP Signature:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc1/apache-omid-incubating-0.9.0.0-src.tar.gz.asc
>
>
> MD5 Hash:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.
> 0-rc1/apache-omid-incubating-0.9.0.0-src.tar.gz.md5
>
>
> Keys to verify the signature of the release artifact are available at:
>
> https://dist.apache.org/repos/dist/dev/incubator/omid/0.9.0.0-rc1/KEYS
>
>
> We request the permission of IPMC to publish the above release candidate as
> Apache Omid 0.9.0.0-incubating. Please try out the package and vote. The
> vote is open for a minimum of 72 hours or until the necessary number of
> votes (3 binding +1s) is reached. [ ] +1 Release this package as Apache
> Omid 0.9.0.0-incubating [ ] 0 I don't feel strongly about it, but I'm okay
> with the release [ ] -1 Do not release this package because... Please add
> (binding) if your vote is binding.
>
>
> Thanks,
> Ohad
>