Fix failed testcases

kanoshiou · kanoshiou · commit 1646eccc99a4 · 2025-02-14T18:46:44.000+08:00
diff --git a/x-pack/plugin/esql/qa/testFixtures/src/main/resources/floats.csv-spec b/x-pack/plugin/esql/qa/testFixtures/src/main/resources/floats.csv-spec
@@ -11,32 +11,32 @@ inDouble
 from employees | keep emp_no, height, height.float, height.half_float, height.scaled_float | where height in (2.03, 2.0299999713897705, 2.029296875, 2.0300000000000002) | sort emp_no;
 
 emp_no:integer |height:double  |height.float:double |height.half_float:double |height.scaled_float:double
-10001          |2.03           |2.0299999713897705  |2.029296875              |2.0300000000000002
-10090          |2.03           |2.0299999713897705  |2.029296875              |2.0300000000000002
+10001          |2.03           |2.0299999713897705  |2.029296875              |2.03
+10090          |2.03           |2.0299999713897705  |2.029296875              |2.03
 ;
 
 inFloat
 from employees | keep emp_no, height, height.float, height.half_float, height.scaled_float | where height.float in (2.03, 2.0299999713897705, 2.029296875, 2.0300000000000002) | sort emp_no;
 
 emp_no:integer |height:double  |height.float:double |height.half_float:double |height.scaled_float:double
-10001          |2.03           |2.0299999713897705  |2.029296875              |2.0300000000000002
-10090          |2.03           |2.0299999713897705  |2.029296875              |2.0300000000000002
+10001          |2.03           |2.0299999713897705  |2.029296875              |2.03
+10090          |2.03           |2.0299999713897705  |2.029296875              |2.03
 ;
 
 inHalfFloat
 from employees | keep emp_no, height, height.float, height.half_float, height.scaled_float | where height.half_float in (2.03, 2.0299999713897705, 2.029296875, 2.0300000000000002) | sort emp_no;
 
 emp_no:integer |height:double  |height.float:double |height.half_float:double |height.scaled_float:double
-10001          |2.03           |2.0299999713897705  |2.029296875              |2.0300000000000002
-10090          |2.03           |2.0299999713897705  |2.029296875              |2.0300000000000002
+10001          |2.03           |2.0299999713897705  |2.029296875              |2.03
+10090          |2.03           |2.0299999713897705  |2.029296875              |2.03
 ;
 
 inScaledFloat
 from employees | keep emp_no, height, height.float, height.half_float, height.scaled_float | where height.scaled_float in (2.03, 2.0299999713897705, 2.029296875, 2.0300000000000002) | sort emp_no;
 
 emp_no:integer |height:double  |height.float:double |height.half_float:double |height.scaled_float:double
-10001          |2.03           |2.0299999713897705  |2.029296875              |2.0300000000000002
-10090          |2.03           |2.0299999713897705  |2.029296875              |2.0300000000000002
+10001          |2.03           |2.0299999713897705  |2.029296875              |2.03
+10090          |2.03           |2.0299999713897705  |2.029296875              |2.03
 ;
 
 convertFromDouble
diff --git a/x-pack/plugin/esql/qa/testFixtures/src/main/resources/keep.csv-spec b/x-pack/plugin/esql/qa/testFixtures/src/main/resources/keep.csv-spec
@@ -269,7 +269,7 @@ simpleEvalWithSortAndLimitOne
 from employees | eval x = languages + 7 | sort x, avg_worked_seconds | limit 1;
 
 avg_worked_seconds:long | birth_date:date | emp_no:integer | first_name:keyword | gender:keyword | height:double | height.float:double | height.half_float:double | height.scaled_float:double | hire_date:date | is_rehired:boolean | job_positions:keyword | languages:integer | languages.byte:integer | languages.long:long | languages.short:integer | last_name:keyword | salary:integer | salary_change:double | salary_change.int:integer | salary_change.keyword:keyword |salary_change.long:long | still_hired:boolean | x:integer
-208374744         |1956-11-14T00:00:00.000Z|10033          |null           |M              |1.63           |1.6299999952316284|1.6298828125     |1.6300000000000001 |1987-03-18T00:00:00.000Z|true           |null           |1              |1              |1              |1              |Merlo          |70011          |null           |null             |null                 |null              |false          |8                                          
+208374744         |1956-11-14T00:00:00.000Z|10033          |null           |M              |1.63           |1.6299999952316284|1.6298828125     |1.63        |1987-03-18T00:00:00.000Z|true           |null           |1              |1              |1              |1              |Merlo          |70011          |null           |null             |null                 |null              |false          |8                                          
 ;
 
 evalOfAverageValue
diff --git a/x-pack/plugin/esql/qa/testFixtures/src/main/resources/mv_expand.csv-spec b/x-pack/plugin/esql/qa/testFixtures/src/main/resources/mv_expand.csv-spec
@@ -322,7 +322,7 @@ keepStarMvExpand#[skip:-8.12.99]
 from employees | where emp_no == 10001 | keep * |  mv_expand first_name;
 
 avg_worked_seconds:long | birth_date:date          | emp_no:integer | first_name:keyword | gender:keyword | height:double | height.float:double | height.half_float:double | height.scaled_float:double | hire_date:date           | is_rehired:boolean | job_positions:keyword                 | languages:integer | languages.byte:integer | languages.long:long | languages.short:integer | last_name:keyword | salary:integer | salary_change:double | salary_change.int:integer | salary_change.keyword:keyword | salary_change.long:long | still_hired:boolean  
-268728049               | 1953-09-02T00:00:00.000Z | 10001          | Georgi             | M              | 2.03          | 2.0299999713897705  | 2.029296875              | 2.0300000000000002         | 1986-06-26T00:00:00.000Z | [false, true]      | [Accountant, Senior Python Developer] | 2                 | 2                      | 2                   | 2                       | Facello           | 57305          | 1.19                 | 1                         | 1.19                          | 1                       | true           
+268728049               | 1953-09-02T00:00:00.000Z | 10001          | Georgi             | M              | 2.03          | 2.0299999713897705  | 2.029296875              | 2.03                       | 1986-06-26T00:00:00.000Z | [false, true]      | [Accountant, Senior Python Developer] | 2                 | 2                      | 2                   | 2                       | Facello           | 57305          | 1.19                 | 1                         | 1.19                          | 1                       | true           
 ;
 
 
diff --git a/x-pack/plugin/esql/qa/testFixtures/src/main/resources/rename.csv-spec b/x-pack/plugin/esql/qa/testFixtures/src/main/resources/rename.csv-spec
@@ -163,7 +163,7 @@ FROM employees | eval x = salary | rename x as y | rename y as x | sort x | rena
 
 avg_worked_seconds:l | birth_date:date          | emp_no:i             | first_name:s         | gender:s             | height:d             | height.float:d       | height.half_float:d  | height.scaled_float:d| hire_date:date           | is_rehired:bool      | job_positions:s      | languages:i          | languages.byte:i     | languages.long:l     | languages.short:i    | last_name:s          | salary:i             | salary_change:d      | salary_change.int:i  | salary_change.keyword:s | salary_change.long:l | still_hired:bool  | y:i          
 
-390266432            | 1959-08-19T00:00:00.000Z | 10015                | Guoxiang             | null                 | 1.66                 | 1.659999966621399    | 1.66015625           | 1.6600000000000001   | 1987-07-02T00:00:00.000Z | [false, false, false, true]| [Head Human Resources, Junior Developer, Principal Support Engineer, Support Engineer]  | 5                    | 5                    | 5                    | 5                    | Nooteboom            | 25324                | [12.4, 14.25]               | [12, 14]             | [12.40, 14.25]             | [12, 14]             | true                 | 25324               
+390266432            | 1959-08-19T00:00:00.000Z | 10015                | Guoxiang             | null                 | 1.66                 | 1.659999966621399    | 1.66015625           | 1.66                 | 1987-07-02T00:00:00.000Z | [false, false, false, true]| [Head Human Resources, Junior Developer, Principal Support Engineer, Support Engineer]  | 5                    | 5                    | 5                    | 5                    | Nooteboom            | 25324                | [12.4, 14.25]               | [12, 14]             | [12.40, 14.25]             | [12, 14]             | true                 | 25324               
 203838153            | 1953-02-08T00:00:00.000Z | 10035                | null                 | M                    | 1.81                 | 1.809999942779541    | 1.8095703125         | 1.81                 | 1988-09-05T00:00:00.000Z | false                      | [Data Scientist, Senior Python Developer]                                               | 5                    | 5                    | 5                    | 5                    | Chappelet            | 25945                | [-6.58, -2.54]              | [-6, -2]             | [-2.54, -6.58]             | [-6, -2]             | false                | 25945               
 313407352            | 1964-10-18T00:00:00.000Z | 10092                | Valdiodio            | F                    | 1.75                 | 1.75                 | 1.75                 | 1.75                 | 1989-09-22T00:00:00.000Z | [false, false, true, true] | [Accountant, Junior Developer]                                                          | 1                    | 1                    | 1                    | 1                    | Niizuma              | 25976                | [-6.77, 0.39, 8.3, 8.78]    | [-6, 0, 8, 8]        | [-6.77, 0.39, 8.30,8.78]   | [-6, 0, 8, 8]        | false                | 25976               
 248451647            | null                     | 10048                | Florian              | M                    | 2.0                  | 2.0                  | 2.0                  | 2.0                  | 1985-02-24T00:00:00.000Z | [true, true]               | Internship                                                                              | 3                    | 3                    | 3                    | 3                    | Syrotiuk             | 26436                | null                        | null                 | null                       | null                 | false                | 26436               
diff --git a/x-pack/plugin/esql/qa/testFixtures/src/main/resources/string.csv-spec b/x-pack/plugin/esql/qa/testFixtures/src/main/resources/string.csv-spec
@@ -737,7 +737,7 @@ convertFromFloats
 from employees | sort emp_no| eval double = to_string(height), float = to_string(height.float), scaled_float = to_string(height.scaled_float), half_float = to_string(height.half_float) | keep emp_no, double, float, scaled_float, half_float, height  | limit 2;
 
 emp_no:integer |double:keyword |float:keyword     |scaled_float:keyword   |half_float:keyword   |height:double
-10001          |2.03           |2.0299999713897705|2.0300000000000002     |2.029296875          |2.03
+10001          |2.03           |2.0299999713897705|2.03                   |2.029296875          |2.03
 10002          |2.08           |2.0799999237060547|2.08                   |2.080078125          |2.08
 ;
 

Original file line number	Diff line number	Diff line change
`@@ -269,7 +269,7 @@ simpleEvalWithSortAndLimitOne`
`269`	`269`	`from employees \| eval x = languages + 7 \| sort x, avg_worked_seconds \| limit 1;`
`270`	`270`
`271`	`271`	avg_worked_seconds:long \| birth_date:date \| emp_no:integer \| first_name:keyword \| gender:keyword \| height:double \| height.float:double \| height.half_float:double \| height.scaled_float:double \| hire_date:date \| is_rehired:boolean \| job_positions:keyword \| languages:integer \| languages.byte:integer \| languages.long:long \| languages.short:integer \| last_name:keyword \| salary:integer \| salary_change:double \| salary_change.int:integer \| salary_change.keyword:keyword \|salary_change.long:long \| still_hired:boolean \| x:integer
`272`		`-208374744 \|1956-11-14T00:00:00.000Z\|10033 \|null \|M \|1.63 \|1.6299999952316284\|1.6298828125 \|1.6300000000000001 \|1987-03-18T00:00:00.000Z\|true \|null \|1 \|1 \|1 \|1 \|Merlo \|70011 \|null \|null \|null \|null \|false \|8`
	`272`	`+208374744 \|1956-11-14T00:00:00.000Z\|10033 \|null \|M \|1.63 \|1.6299999952316284\|1.6298828125 \|1.63 \|1987-03-18T00:00:00.000Z\|true \|null \|1 \|1 \|1 \|1 \|Merlo \|70011 \|null \|null \|null \|null \|false \|8`
`273`	`273`	`;`
`274`	`274`
`275`	`275`	`evalOfAverageValue`
Original file line number	Diff line number	Diff line change
`@@ -322,7 +322,7 @@ keepStarMvExpand#[skip:-8.12.99]`
`322`	`322`	`from employees \| where emp_no == 10001 \| keep * \| mv_expand first_name;`
`323`	`323`
`324`	`324`	avg_worked_seconds:long \| birth_date:date \| emp_no:integer \| first_name:keyword \| gender:keyword \| height:double \| height.float:double \| height.half_float:double \| height.scaled_float:double \| hire_date:date \| is_rehired:boolean \| job_positions:keyword \| languages:integer \| languages.byte:integer \| languages.long:long \| languages.short:integer \| last_name:keyword \| salary:integer \| salary_change:double \| salary_change.int:integer \| salary_change.keyword:keyword \| salary_change.long:long \| still_hired:boolean
`325`		-268728049 \| 1953-09-02T00:00:00.000Z \| 10001 \| Georgi \| M \| 2.03 \| 2.0299999713897705 \| 2.029296875 \| 2.0300000000000002 \| 1986-06-26T00:00:00.000Z \| [false, true] \| [Accountant, Senior Python Developer] \| 2 \| 2 \| 2 \| 2 \| Facello \| 57305 \| 1.19 \| 1 \| 1.19 \| 1 \| true
	`325`	+268728049 \| 1953-09-02T00:00:00.000Z \| 10001 \| Georgi \| M \| 2.03 \| 2.0299999713897705 \| 2.029296875 \| 2.03 \| 1986-06-26T00:00:00.000Z \| [false, true] \| [Accountant, Senior Python Developer] \| 2 \| 2 \| 2 \| 2 \| Facello \| 57305 \| 1.19 \| 1 \| 1.19 \| 1 \| true
`326`	`326`	`;`
`327`	`327`
`328`	`328`
Original file line number	Diff line number	Diff line change
`@@ -737,7 +737,7 @@ convertFromFloats`
`737`	`737`	`from employees \| sort emp_no\| eval double = to_string(height), float = to_string(height.float), scaled_float = to_string(height.scaled_float), half_float = to_string(height.half_float) \| keep emp_no, double, float, scaled_float, half_float, height \| limit 2;`
`738`	`738`
`739`	`739`	`emp_no:integer \|double:keyword \|float:keyword \|scaled_float:keyword \|half_float:keyword \|height:double`
`740`		`-10001 \|2.03 \|2.0299999713897705\|2.0300000000000002 \|2.029296875 \|2.03`
	`740`	`+10001 \|2.03 \|2.0299999713897705\|2.03 \|2.029296875 \|2.03`
`741`	`741`	`10002 \|2.08 \|2.0799999237060547\|2.08 \|2.080078125 \|2.08`
`742`	`742`	`;`
`743`	`743`