[query] Move LoweredTableReaderCoercer into ExecuteContext - Githubissues

hail-is / hail

Cloud-native genomic dataframes and batch computing

https://hail.is

MIT License

974 stars 243 forks source link

[query] Move LoweredTableReaderCoercer into ExecuteContext #14696

Open ehigham opened 1 week ago

ehigham commented 1 week ago

Refactored table reader coercion and caching mechanism.

What changed?

Removed shouldCacheQueryInfo method from Backend class
Introduced CoercerCache in ExecuteContext
Refactored LoweredTableReader.makeCoercer to return a function instead of a class
Removed local caching in GenericTableValue and LoweredTableReader
Added NoCaching utility

Why make this change?

This change aims to optimize table reader coercion by:

Centralizing caching logic in ExecuteContext
Allowing more flexible caching strategies across different backend implementations

ehigham commented 1 week ago

[!WARNING] This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite. Learn more

#14698
#14696 👈
#14693
#14692
#14691
#14690
#14686
#14684
#14683
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ehigham and the rest of your teammates on Graphite