[ 
https://issues.apache.org/jira/browse/DERBY-2599?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12523274
 ] 

Mamta A. Satoor commented on DERBY-2599:
----------------------------------------

Made a commit in main codeline which just fixes the comments. The revision for 
change is 570482. I also migrated this change into 10.3 codeline with revision 
570483.

Commit comments are as follows.

Fixed the comments for getDominantTypeServices() method in ValueNodeList. The 
comments were using an example of concatenation operation but concatenation 
operation does not call this method to find the dominant type of it's 2 
operands. Instead, concatenation operator implements it's own simple algorithm 
in ConcatenationOperatorNode. This is because ConcatenationOperatorNode does 
not work off of a ValueNodeList. I have reworded the comments for 
getDominantTypeServices() to describe what is doing.

> Set correct collation type and derivation on DataTypeDescriptor(DTD).
> ---------------------------------------------------------------------
>
>                 Key: DERBY-2599
>                 URL: https://issues.apache.org/jira/browse/DERBY-2599
>             Project: Derby
>          Issue Type: New Feature
>          Components: SQL
>    Affects Versions: 10.3.1.4
>            Reporter: Mamta A. Satoor
>            Assignee: Mamta A. Satoor
>             Fix For: 10.3.1.4
>
>         Attachments: DERBY2599_collationType_default_UCS_BASIC_v1_diff.txt, 
> DERBY2599_collationType_default_UCS_BASIC_v1_stat.txt, 
> DERBY2599_correct_collation_for_cast_v1_diff.txt, 
> DERBY2599_correct_collation_for_cast_v1_stat.txt, 
> DERBY2599_getNull_should_set_collation_info_v1_diff.txt, 
> DERBY2599_getNull_should_set_collation_info_v1_stat.txt, 
> DERBY2599_IntermediatePatch_v1_diff.txt, 
> DERBY2599_IntermediatePatch_v1_stat.txt, 
> DERBY2599_Set_collation_for_aggregates_v1_diff.txt, 
> DERBY2599_Set_collation_for_aggregates_v1_stat.txt, 
> DERBY2599_Set_collation_for_aggregates_v1_stat.txt, 
> DERBY2599_Set_collation_for_Max_Min_v1_diff.txt, 
> DERBY2599_set_correct_collation_for_params_and_string_functions_v1_diff.txt, 
> DERBY2599_set_correct_collation_for_params_and_string_functions_v1_stat.txt, 
> DERBY2599_Use_collation_of_compilation_schema_and_not_current_schema_diff_v1.txt,
>  
> DERBY2599_Use_collation_of_compilation_schema_and_not_current_schema_stat_v1.txt
>
>
> DTD has TypeDescriptorImpl in it which has 2 new fields, namely, 
> collationType and collationDerivation. These 2 fields are available for all 
> different types of DTDs but only apply to character types. The other 
> datatypes should ignore these 2 fields.
> This Jira is a placeholder for loading the correct values into collationType 
> and collationDerivation.

-- 
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