ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Pavel Tupitsyn <ptupit...@apache.org>
Subject Re: Add IgniteInvalidGridStateException instead of IllegalStateException in KernalGateway
Date Mon, 31 Aug 2020 19:45:02 GMT
Actually, we already have IgniteIllegalStateException,
it exists exactly for situations like this.

Let's just change GridKernalGatewayImpl to throw IgniteIllegalStateException
instead of IllegalStateException and handle that in .NET

On Sat, Aug 1, 2020 at 10:32 AM Pavel Tupitsyn <ptupitsyn@apache.org> wrote:

> Sounds good to me.
> We can include grid state as a property of that exception too.
>
> On Fri, Jul 31, 2020 at 5:32 PM Guru Stron <gurustronpublic@gmail.com>
> wrote:
>
>> Dear Igniters,
>>
>> I'm working on a bug in .NET code that handles transaction states (jira
>> ticket <https://issues.apache.org/jira/browse/IGNITE-13296>). And I need
>> a
>> way to handle stopped grid exception, so I think it will make sense to
>> throw a specialized exception type in this case instead of a generic
>> IllegalStateException
>> <
>> https://github.com/apache/ignite/blob/d486f24b1be3f08e8b1e5e9992c4c8718e169299/modules/core/src/main/java/org/apache/ignite/internal/GridKernalGatewayImpl.java#L95
>> >
>> .
>>
>> Any objections?
>>
>> Best regards!
>>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message