[ 
https://issues.apache.org/jira/browse/DERBY-3310?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12571647#action_12571647
 ] 

Daniel John Debrunner commented on DERBY-3310:
----------------------------------------------

Looking at NormalizeResultSet I now see that cast nodes are not needed. 
NormalizeResultSet performs type
conversions itself without any need for cast nodes.

It needs a set of DataTypeDescriptors that represent the intended types. SO I'm 
guessing the layout should be
something like to say convert our BIGINT into an INT

NormalizeNode
    ResultColumn(BIGINT) -> VirtualColumnNode (??) -> A (below)

    source result set node
         A - ResultColumn (INT) -> constant (INT)


That's a rough guess, but I think it's beginning to show why the fix works. I 
think without the change
the type of A would be set to BIGINT and with the change it would remain at INT.

Not saying I fully understand it yet, but just throwing more info out there 
from looking at the code.


  

> ASSERT in MergeSort.checkColumnTypes() disallow legal type conversions
> ----------------------------------------------------------------------
>
>                 Key: DERBY-3310
>                 URL: https://issues.apache.org/jira/browse/DERBY-3310
>             Project: Derby
>          Issue Type: Bug
>          Components: SQL
>    Affects Versions: 10.4.0.0
>            Reporter: Dyre Tjeldvoll
>            Assignee: Kathey Marsden
>            Priority: Minor
>         Attachments: cast-repro.sql, derby-3310_try1_diff.txt
>
>
> The following code 
> CREATE TABLE U (SNAME VARCHAR(32000), TNAME VARCHAR(32000), C1 BIGINT);
> -- This triggers an ASSERT (because 2 is INTEGER and not BIGINT)
> INSERT INTO U(SNAME, TNAME, C1) SELECT DISTINCT SCHEMANAME, TABLENAME, 2
>  FROM SYS.SYSTABLES T JOIN SYS.SYSSCHEMAS S ON T.SCHEMAID = S.SCHEMAID;
> gives
> ERROR XJ001: Java exception: 'ASSERT FAILED col1.getClass() (class 
> org.apache.derby.iapi.types.SQLInteger) expected to be the same as 
> col2.getClass() (class org.apache.derby.iapi.types.SQLLongint): 
> org.apache.derby.shared.common.sanity.AssertFailure'.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to