[IGNITE-9836] Invalid check of ea java versions

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[IGNITE-9836] Invalid check of ea java versions

sdarlington
Hi,

Would someone mind taking a quick look at this ticket? Basically, a clean download of Ignite won’t start if the version of Java you’re using has a number like “java version "1.8.0_202-ea””. (This is the default if you get your JDK using Homebrew on a Mac.)

> https://issues.apache.org/jira/browse/IGNITE-9836 <https://issues.apache.org/jira/browse/IGNITE-9836>

This has been bugging me for ages and now that I look at it I find that there’s already a tiny, working patch available.

Regards,
Stephen
Reply | Threaded
Open this post in threaded view
|

Re: [IGNITE-9836] Invalid check of ea java versions

dmagda
Stephen,

Thanks for sending the patch! Seems that Igniters are already actively
reviewing it in JIRA.

-
Denis


On Mon, Sep 16, 2019 at 7:03 AM Stephen Darlington <
[hidden email]> wrote:

> Hi,
>
> Would someone mind taking a quick look at this ticket? Basically, a clean
> download of Ignite won’t start if the version of Java you’re using has a
> number like “java version "1.8.0_202-ea””. (This is the default if you get
> your JDK using Homebrew on a Mac.)
>
> > https://issues.apache.org/jira/browse/IGNITE-9836 <
> https://issues.apache.org/jira/browse/IGNITE-9836>
>
> This has been bugging me for ages and now that I look at it I find that
> there’s already a tiny, working patch available.
>
> Regards,
> Stephen
Reply | Threaded
Open this post in threaded view
|

Re: [IGNITE-9836] Invalid check of ea java versions

sdarlington
I can’t take any credit for the patch but if the original author has lost interest I’m happy to help push it through.

Regards,
Stephen

> On 16 Sep 2019, at 19:31, Denis Magda <[hidden email]> wrote:
>
> Stephen,
>
> Thanks for sending the patch! Seems that Igniters are already actively
> reviewing it in JIRA.
>
> -
> Denis
>
>
> On Mon, Sep 16, 2019 at 7:03 AM Stephen Darlington <
> [hidden email]> wrote:
>
>> Hi,
>>
>> Would someone mind taking a quick look at this ticket? Basically, a clean
>> download of Ignite won’t start if the version of Java you’re using has a
>> number like “java version "1.8.0_202-ea””. (This is the default if you get
>> your JDK using Homebrew on a Mac.)
>>
>>> https://issues.apache.org/jira/browse/IGNITE-9836 <
>> https://issues.apache.org/jira/browse/IGNITE-9836>
>>
>> This has been bugging me for ages and now that I look at it I find that
>> there’s already a tiny, working patch available.
>>
>> Regards,
>> Stephen


Reply | Threaded
Open this post in threaded view
|

Re: [IGNITE-9836] Invalid check of ea java versions

Ilya Kasnacheev
Hello!

Please do!

Regards,
--
Ilya Kasnacheev


вт, 17 сент. 2019 г. в 11:13, Stephen Darlington <
[hidden email]>:

> I can’t take any credit for the patch but if the original author has lost
> interest I’m happy to help push it through.
>
> Regards,
> Stephen
>
> > On 16 Sep 2019, at 19:31, Denis Magda <[hidden email]> wrote:
> >
> > Stephen,
> >
> > Thanks for sending the patch! Seems that Igniters are already actively
> > reviewing it in JIRA.
> >
> > -
> > Denis
> >
> >
> > On Mon, Sep 16, 2019 at 7:03 AM Stephen Darlington <
> > [hidden email]> wrote:
> >
> >> Hi,
> >>
> >> Would someone mind taking a quick look at this ticket? Basically, a
> clean
> >> download of Ignite won’t start if the version of Java you’re using has a
> >> number like “java version "1.8.0_202-ea””. (This is the default if you
> get
> >> your JDK using Homebrew on a Mac.)
> >>
> >>> https://issues.apache.org/jira/browse/IGNITE-9836 <
> >> https://issues.apache.org/jira/browse/IGNITE-9836>
> >>
> >> This has been bugging me for ages and now that I look at it I find that
> >> there’s already a tiny, working patch available.
> >>
> >> Regards,
> >> Stephen
>
>
>
Reply | Threaded
Open this post in threaded view
|

Re: [IGNITE-9836] Invalid check of ea java versions

sdarlington
Done: https://github.com/apache/ignite/pull/6920

While we’re talking about the startup scripts… https://issues.apache.org/jira/browse/IGNITE-11856

Regards,
Stephen

> On 26 Sep 2019, at 17:02, Ilya Kasnacheev <[hidden email]> wrote:
>
> Hello!
>
> Please do!
>
> Regards,
> --
> Ilya Kasnacheev
>
>
> вт, 17 сент. 2019 г. в 11:13, Stephen Darlington <
> [hidden email]>:
>
>> I can’t take any credit for the patch but if the original author has lost
>> interest I’m happy to help push it through.
>>
>> Regards,
>> Stephen
>>
>>> On 16 Sep 2019, at 19:31, Denis Magda <[hidden email]> wrote:
>>>
>>> Stephen,
>>>
>>> Thanks for sending the patch! Seems that Igniters are already actively
>>> reviewing it in JIRA.
>>>
>>> -
>>> Denis
>>>
>>>
>>> On Mon, Sep 16, 2019 at 7:03 AM Stephen Darlington <
>>> [hidden email]> wrote:
>>>
>>>> Hi,
>>>>
>>>> Would someone mind taking a quick look at this ticket? Basically, a
>> clean
>>>> download of Ignite won’t start if the version of Java you’re using has a
>>>> number like “java version "1.8.0_202-ea””. (This is the default if you
>> get
>>>> your JDK using Homebrew on a Mac.)
>>>>
>>>>> https://issues.apache.org/jira/browse/IGNITE-9836 <
>>>> https://issues.apache.org/jira/browse/IGNITE-9836>
>>>>
>>>> This has been bugging me for ages and now that I look at it I find that
>>>> there’s already a tiny, working patch available.
>>>>
>>>> Regards,
>>>> Stephen
>>
>>
>>


Reply | Threaded
Open this post in threaded view
|

Re: [IGNITE-9836] Invalid check of ea java versions

Ilya Kasnacheev
Hello!

Thank you, I have merged both.

Regards,
--
Ilya Kasnacheev


пт, 27 сент. 2019 г. в 14:36, Stephen Darlington <
[hidden email]>:

> Done: https://github.com/apache/ignite/pull/6920
>
> While we’re talking about the startup scripts…
> https://issues.apache.org/jira/browse/IGNITE-11856
>
> Regards,
> Stephen
>
> > On 26 Sep 2019, at 17:02, Ilya Kasnacheev <[hidden email]>
> wrote:
> >
> > Hello!
> >
> > Please do!
> >
> > Regards,
> > --
> > Ilya Kasnacheev
> >
> >
> > вт, 17 сент. 2019 г. в 11:13, Stephen Darlington <
> > [hidden email]>:
> >
> >> I can’t take any credit for the patch but if the original author has
> lost
> >> interest I’m happy to help push it through.
> >>
> >> Regards,
> >> Stephen
> >>
> >>> On 16 Sep 2019, at 19:31, Denis Magda <[hidden email]> wrote:
> >>>
> >>> Stephen,
> >>>
> >>> Thanks for sending the patch! Seems that Igniters are already actively
> >>> reviewing it in JIRA.
> >>>
> >>> -
> >>> Denis
> >>>
> >>>
> >>> On Mon, Sep 16, 2019 at 7:03 AM Stephen Darlington <
> >>> [hidden email]> wrote:
> >>>
> >>>> Hi,
> >>>>
> >>>> Would someone mind taking a quick look at this ticket? Basically, a
> >> clean
> >>>> download of Ignite won’t start if the version of Java you’re using
> has a
> >>>> number like “java version "1.8.0_202-ea””. (This is the default if you
> >> get
> >>>> your JDK using Homebrew on a Mac.)
> >>>>
> >>>>> https://issues.apache.org/jira/browse/IGNITE-9836 <
> >>>> https://issues.apache.org/jira/browse/IGNITE-9836>
> >>>>
> >>>> This has been bugging me for ages and now that I look at it I find
> that
> >>>> there’s already a tiny, working patch available.
> >>>>
> >>>> Regards,
> >>>> Stephen
> >>
> >>
> >>
>
>
>