Skip to content

[query] remove non-primitives from scala fs - #15178

Open
ehigham wants to merge 1 commit into
mainfrom
ehigham/scala-fs-extract-primitives
Open

[query] remove non-primitives from scala fs#15178
ehigham wants to merge 1 commit into
mainfrom
ehigham/scala-fs-extract-primitives

Conversation

@ehigham

@ehigham ehigham commented Nov 5, 2025

Copy link
Copy Markdown
Member

Moved utility functions from the FS trait to a package object, reducing the size of the trait. Converted methods like getCodecFromPath, getExtension, and getCodecExtension into standalone functions. Also improved error handling by using Option for codec returns instead of nullable values, and implemented pattern matching for cleaner code.

This change cannot impact the Hail Batch instance as deployed by Broad Institute in GCP

ehigham commented Nov 5, 2025

Copy link
Copy Markdown
Member Author

@ehigham
ehigham force-pushed the ehigham/scala-fs-cleanup branch from 38bd775 to 7ea3e1a Compare November 5, 2025 19:13
@ehigham
ehigham force-pushed the ehigham/scala-fs-extract-primitives branch from ac8c10a to 63d9631 Compare November 5, 2025 19:13
@ehigham
ehigham marked this pull request as ready for review November 8, 2025 02:16
@ehigham
ehigham force-pushed the ehigham/scala-fs-cleanup branch from 7ea3e1a to 7bdccf4 Compare November 13, 2025 20:57
@ehigham
ehigham force-pushed the ehigham/scala-fs-extract-primitives branch from 63d9631 to 351551b Compare November 13, 2025 20:57
Base automatically changed from ehigham/scala-fs-cleanup to main November 14, 2025 03:47
@ehigham
ehigham force-pushed the ehigham/scala-fs-extract-primitives branch 3 times, most recently from 05617b0 to 6079d3d Compare November 18, 2025 03:48
@ehigham
ehigham force-pushed the ehigham/scala-fs-extract-primitives branch from 6079d3d to 0bcda90 Compare November 20, 2025 04:01
@ehigham
ehigham force-pushed the ehigham/scala-fs-extract-primitives branch from 0bcda90 to a460934 Compare January 5, 2026 18:04
@ehigham
ehigham force-pushed the ehigham/scala-fs-extract-primitives branch from a460934 to 8909888 Compare February 19, 2026 15:14
@ehigham
ehigham force-pushed the ehigham/scala-fs-extract-primitives branch from 8909888 to 1af1120 Compare February 19, 2026 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant