Skip to content

Conversation

@LuciferYang
Copy link
Contributor

@LuciferYang LuciferYang commented Apr 28, 2025

What changes were proposed in this pull request?

This pr aims to Re-generate golden files of postgreSQL/float4.sql and postgreSQL/int8.sql for Java 21. #50604 forgot to generate them.

Why are the changes needed?

Restore Java 21 daily test:

image

image

Does this PR introduce any user-facing change?

No

How was this patch tested?

  • Pass Github Actions
  • locally test with Java 21

Was this patch authored or co-authored using generative AI tooling?

No

@github-actions github-actions bot added the SQL label Apr 28, 2025
@LuciferYang LuciferYang changed the title [SPARK-51817][SQL][TESTS] Re-generate golden files for Java 21 [SPARK-51817][SQL][TESTS][FOLLOWUP] Re-generate golden files for Java 21 Apr 28, 2025
@LuciferYang LuciferYang changed the title [SPARK-51817][SQL][TESTS][FOLLOWUP] Re-generate golden files for Java 21 [SPARK-51937][SQL][TESTS] Re-generate golden files of postgreSQL/float4.sql and postgreSQL/int8.sql for Java 21. Apr 28, 2025
@LuciferYang LuciferYang changed the title [SPARK-51937][SQL][TESTS] Re-generate golden files of postgreSQL/float4.sql and postgreSQL/int8.sql for Java 21. [SPARK-51937][SQL][TESTS] Re-generate golden files of postgreSQL/float4.sql and postgreSQL/int8.sql for Java 21 Apr 28, 2025
@LuciferYang
Copy link
Contributor Author

LuciferYang commented Apr 28, 2025

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1, LGTM. Thank you, @LuciferYang .

LuciferYang added a commit that referenced this pull request Apr 28, 2025
…at4.sql` and `postgreSQL/int8.sql` for Java 21

### What changes were proposed in this pull request?
This pr aims to Re-generate golden files of `postgreSQL/float4.sql` and `postgreSQL/int8.sql` for Java 21. #50604 forgot to generate them.

### Why are the changes needed?
Restore Java 21 daily test:
- master: https://github.com/apache/spark/actions/runs/14699959163/job/41247643789
- branch-4.0: https://github.com/apache/spark/actions/runs/14700610338/job/41249373892

![image](https://github.com/user-attachments/assets/4bfe3b14-d5a8-4428-928b-8a2e94287bb0)

![image](https://github.com/user-attachments/assets/ab725b03-2da6-4460-8cc7-759806965fbd)

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
- Pass Github Actions
- locally test with Java 21

### Was this patch authored or co-authored using generative AI tooling?
No

Closes #50740 from LuciferYang/SPARK-51817-FOLLOWUP.

Authored-by: yangjie01 <yangjie01@baidu.com>
Signed-off-by: yangjie01 <yangjie01@baidu.com>
(cherry picked from commit 6c37ee4)
Signed-off-by: yangjie01 <yangjie01@baidu.com>
@LuciferYang
Copy link
Contributor Author

Merged into master/branch-4.0 to restore Java 21 daily test. Thanks @dongjoon-hyun

@LuciferYang LuciferYang deleted the SPARK-51817-FOLLOWUP branch May 1, 2025 13:39
yhuang-db pushed a commit to yhuang-db/spark that referenced this pull request Jun 9, 2025
…at4.sql` and `postgreSQL/int8.sql` for Java 21

### What changes were proposed in this pull request?
This pr aims to Re-generate golden files of `postgreSQL/float4.sql` and `postgreSQL/int8.sql` for Java 21. apache#50604 forgot to generate them.

### Why are the changes needed?
Restore Java 21 daily test:
- master: https://github.com/apache/spark/actions/runs/14699959163/job/41247643789
- branch-4.0: https://github.com/apache/spark/actions/runs/14700610338/job/41249373892

![image](https://github.com/user-attachments/assets/4bfe3b14-d5a8-4428-928b-8a2e94287bb0)

![image](https://github.com/user-attachments/assets/ab725b03-2da6-4460-8cc7-759806965fbd)

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
- Pass Github Actions
- locally test with Java 21

### Was this patch authored or co-authored using generative AI tooling?
No

Closes apache#50740 from LuciferYang/SPARK-51817-FOLLOWUP.

Authored-by: yangjie01 <yangjie01@baidu.com>
Signed-off-by: yangjie01 <yangjie01@baidu.com>
zifeif2 pushed a commit to zifeif2/spark that referenced this pull request Nov 14, 2025
…at4.sql` and `postgreSQL/int8.sql` for Java 21

### What changes were proposed in this pull request?
This pr aims to Re-generate golden files of `postgreSQL/float4.sql` and `postgreSQL/int8.sql` for Java 21. apache#50604 forgot to generate them.

### Why are the changes needed?
Restore Java 21 daily test:
- master: https://github.com/apache/spark/actions/runs/14699959163/job/41247643789
- branch-4.0: https://github.com/apache/spark/actions/runs/14700610338/job/41249373892

![image](https://github.com/user-attachments/assets/4bfe3b14-d5a8-4428-928b-8a2e94287bb0)

![image](https://github.com/user-attachments/assets/ab725b03-2da6-4460-8cc7-759806965fbd)

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
- Pass Github Actions
- locally test with Java 21

### Was this patch authored or co-authored using generative AI tooling?
No

Closes apache#50740 from LuciferYang/SPARK-51817-FOLLOWUP.

Authored-by: yangjie01 <yangjie01@baidu.com>
Signed-off-by: yangjie01 <yangjie01@baidu.com>
(cherry picked from commit fb76a3d)
Signed-off-by: yangjie01 <yangjie01@baidu.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants