[jira] [Comment Edited] (IGNITE-8432) GridSqlQueryParser.java throws null pointer if there is a mismatch between the column name casing

["Pavel Kuznetsov (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/IGNITE-8432?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16710019#comment-16710019 ] 

Pavel Kuznetsov edited comment on IGNITE-8432 at 12/5/18 12:46 PM:
-------------------------------------------------------------------

[~vozerov] Patch affects only tests, so report above looks ok for me.
But that run some time ago, so to be sure, I merged master to my branch and scheduled only SQL: https://ci.ignite.apache.org/viewQueued.html?itemId=2470563 


was (Author: pkouznet):
[~vozerov] Patch affects only tests, so report above looks ok for me.
To be sure, merged master to branch and scheduled only SQL: https://ci.ignite.apache.org/viewQueued.html?itemId=2470563 

> GridSqlQueryParser.java throws null pointer if there is a mismatch between the column name casing
> -------------------------------------------------------------------------------------------------
>
>                 Key: IGNITE-8432
>                 URL: https://issues.apache.org/jira/browse/IGNITE-8432
>             Project: Ignite
>          Issue Type: Improvement
>          Components: sql
>    Affects Versions: 2.4
>            Reporter: Courtney
>            Assignee: Pavel Kuznetsov
>            Priority: Trivial
>
> In the SQL we generate we had a mismatch between column name casing due to unquoted identifiers being upper cased and we were quoting the column name in the column list but not in the primary key list.
> That lead to a null pointer exception which wasn't the most obvious thing to trace because it was being caught, wrapped and re-thrown.
> Line 1043 of GridSqlQueryParser.java has an assert which leads to the error.
>  



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)