[
https://issues.apache.org/jira/browse/IGNITE-8765?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16691142#comment-16691142
]
ASF GitHub Bot commented on IGNITE-8765:
----------------------------------------
GitHub user pavel-kuznetsov opened a pull request:
https://github.com/apache/ignite/pull/5421
IGNITE-8765: Fixed event's query type
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/gridgain/apache-ignite ignite-8765
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/ignite/pull/5421.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #5421
----
commit d4ecd8454a5511c8b85fb3c3ede0072d71b2757f
Author: palmuhal <palmihal@...>
Date: 2018-11-18T23:49:42Z
ignite-8765: Enabled test for the bug.
Bug itself have been fixed during IGNITE-10253.
commit 6cad0bb22b2b7a0f3a58486c66d145a39eacb312
Author: palmuhal <palmihal@...>
Date: 2018-11-19T00:17:39Z
ignite-8765: Fixed event type.
Earlier no matter query of what type (SQL or SQL_FIELDS) got executed,
"SQL" used to be the query type of the event. Now we make a difference.
Also event message is updated.
----
> SQL event is not fired when query is reduced to local form
> ----------------------------------------------------------
>
> Key: IGNITE-8765
> URL: https://issues.apache.org/jira/browse/IGNITE-8765
> Project: Ignite
> Issue Type: Bug
> Components: sql
> Affects Versions: 2.4, 2.5
> Reporter: Vladimir Ozerov
> Assignee: Pavel Kuznetsov
> Priority: Major
>
> *Reproducer*
> {{org.apache.ignite.internal.processors.cache.IgniteCacheAbstractQuerySelfTest#testSqlQueryEvents}}
> *Root Cause*
> Local and non-local query execution is performed differently. Local query is
> logged early. However, non-local query could be reduced after that. As a
> result, event is missed. Try change
> {{org.apache.ignite.internal.processors.query.h2.sql.GridSqlQueryParser#isLocalQuery}}
> method to always return {{false}} and then re-run failed test to see the
> difference.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
