Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[hxb] only load up to EOF if at least one field is accessed #11927

Draft
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

kLabz
Copy link
Contributor

@kLabz kLabz commented Jan 20, 2025

Trying again with the following from #11866:

  • Similar to [hxb] Reader: lazy type restoring  #11873, uses TLazy on cf_type. Initially, modules that are non essential are only partially loaded. cf_type is being followed when a field is used, which will trigger the loading of the rest of the module.

Will need to list all the things that were wrong with that in #11866, and check if latest fixes there are enough to make this work (issue could have just been #11928).

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