ignite-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Saikat Maitra <saikat.mai...@gmail.com>
Subject Re: SQL query timeout: in progress or abandoned
Date Sun, 18 Aug 2019 01:30:55 GMT
Hi Ivan, Denis

Thank you for your feedback, I am looking into the changes needed for this
issue.

I am also looking into these configurations parameters
https://apacheignite.readme.io/v2.2/docs/configuration-parameters to see if
there are similar attributes being used in  SqlFieldsQuery and SqlQuery.


Regards,

Saikat



On Thu, Aug 15, 2019 at 6:13 AM Павлухин Иван <vololo100@gmail.com> wrote:

> Saikat, Denis,
>
> I left comments in the ticket [1].
>
> [1] https://issues.apache.org/jira/browse/IGNITE-7285
>
> вт, 13 авг. 2019 г. в 21:53, Denis Magda <dmagda@apache.org>:
> >
> > Hi Saikat,
> >
> > Thanks for a quick turnaround! Ivan, could you please step in and do a
> > review?
> >
> > -
> > Denis
> >
> >
> > On Sun, Aug 11, 2019 at 6:26 AM Saikat Maitra <saikat.maitra@gmail.com>
> > wrote:
> >
> > > Hi Denis, Ivan
> > >
> > > As discussed I have updated the PR and incorporated review comments.
> > >
> > > https://github.com/apache/ignite/pull/6490/files
> > >
> > > Please take a look and share your feedback.
> > >
> > > Regard,
> > > Saikat
> > >
> > >
> > >
> > > On Sat, Aug 10, 2019 at 5:51 PM Saikat Maitra <saikat.maitra@gmail.com
> >
> > > wrote:
> > >
> > > > Hello Denis, Ivan
> > > >
> > > > Yes, I can take up the changes for IGNITE-7825.
> > > >
> > > > I had a doubt on the usage of the Default Query Timeout.
> > > >
> > > > I had raised the PR in an assumption that Default Query Timeout will
> only
> > > > be used if user had not provided Cache Query Timeout
> > > >
> > > > https://github.com/apache/ignite/pull/6490/files
> > > >
> > > > I wanted to discuss if it is correct intended usage of Default Query
> > > > Timeout or should we reconsider?
> > > >
> > > > Regards,
> > > > Saikat
> > > >
> > > >
> > > >
> > > > On Fri, Aug 9, 2019 at 12:11 PM Denis Magda <dmagda@apache.org>
> wrote:
> > > >
> > > >> Ivan, thanks for sharing this discussion. Let's use it for our
> > > >> conversation.
> > > >>
> > > >> -
> > > >> Denis
> > > >>
> > > >>
> > > >> On Thu, Aug 8, 2019 at 11:15 PM Павлухин Иван <vololo100@gmail.com>
> > > >> wrote:
> > > >>
> > > >> > Just for the protocol. There was an original dev-list discussion
> [1].
> > > >> > Added a link to the ticket as well.
> > > >> >
> > > >> > [1]
> > > >> >
> > > >>
> > >
> http://apache-ignite-developers.2346864.n4.nabble.com/IGNITE-7285-Add-default-query-timeout-td41828.html
> > > >> >
> > > >> > пт, 9 авг. 2019 г. в 01:22, Denis Magda <dmagda@apache.org>:
> > > >> > >
> > > >> > > Hey Saikat,
> > > >> > >
> > > >> > > Are you still working on this ticket?
> > > >> > > https://issues.apache.org/jira/browse/IGNITE-7285
> > > >> > >
> > > >> > > Seems that's the last API that doesn't support timeouts
- JDBC
> and
> > > >> ODBC
> > > >> > > drivers already go with it.
> > > >> > >
> > > >> > > If you don't have time to complete the changes then someone
else
> > > from
> > > >> the
> > > >> > > community can take over. We see a lot of demand for this
API and
> > > here
> > > >> is
> > > >> > > one example:
> > > >> > >
> > > >> >
> > > >>
> > >
> https://stackoverflow.com/questions/57275301/how-to-set-a-query-timeout-for-apache-ignite-cache
> > > >> > >
> > > >> > > -
> > > >> > > Denis
> > > >> >
> > > >> >
> > > >> >
> > > >> > --
> > > >> > Best regards,
> > > >> > Ivan Pavlukhin
> > > >> >
> > > >>
> > > >
> > >
>
>
>
> --
> Best regards,
> Ivan Pavlukhin
>

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