[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1628 Please modify .travis.yml as well so that build doesn't occur on Oracle JDK 7. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread harshach
Github user harshach commented on the issue: https://github.com/apache/storm/pull/1628 @HeartSaVioR done. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, o

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1628 +1 for the change but would like to hold this until we finish a formal process. For me changing minimum requirement is an important decision for the project so handling this with only ge

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread harshach
Github user harshach commented on the issue: https://github.com/apache/storm/pull/1628 @HeartSaVioR whats the formal process? did we ever define that. I already started discussion thread. There are 3 pmc's who said +1 and so far there is no -1. Its past 72 hrs as well. --- If your

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1628 @harshach Yes, bylaws states that. http://storm.apache.org/contribute/BYLAWS.html ``` Voting Decisions regarding the project are made by votes on the primary project deve

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/storm/pull/1628 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fe

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread satishd
Github user satishd commented on the issue: https://github.com/apache/storm/pull/1628 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread harshach
Github user harshach commented on the issue: https://github.com/apache/storm/pull/1628 @HeartSaVioR started a thread or we can use the PR votes for consensus among PMC. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1628 @harshach Thanks for the quick follow up. I just voted to VOTE thread. Since we've seen some PMCs' positive feedback I guess it should be reflected to vote result. --- If your project is set up

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-16 Thread arunmahadevan
Github user arunmahadevan commented on the issue: https://github.com/apache/storm/pull/1628 +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the fe

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-17 Thread revans2
Github user revans2 commented on the issue: https://github.com/apache/storm/pull/1628 @HeartSaVioR Technically I don't think the bylaws cover changing minimum requirements beyond a code change. If you want to modify the bylaws in that way I would be happy to support you in this.

[GitHub] storm issue #1628: STORM-2041. Make Java 8 as minimum requirement for 2.0 re...

2016-08-18 Thread HeartSaVioR
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1628 I will just do +1 and go merging since we already see our consensus from multiple places. I don't want to become a lawyer :) so that's enough. --- If your project is set up for it, you can r