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

Move all work for _write_metadata_file into a background thread to avoid building metadata in the main thread. This is not all that costly, but it is O(n) where n is the number of arrays in the tree, so it can start to add up for trees with a lot of parameters. #1001

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

copybara-service[bot]
Copy link

@copybara-service copybara-service bot commented Jul 12, 2024

Move all work for _write_metadata_file into a background thread to avoid building metadata in the main thread. This is not all that costly, but it is O(n) where n is the number of arrays in the tree, so it can start to add up for trees with a lot of parameters.

@copybara-service copybara-service bot changed the title Move all work for _write_metadata_file into a background thread to avoid building metadata in the main thread. This is not all that costly, but it is O(n) where n is the number of arrays in the tree, so it can start to add up. Move all work for _write_metadata_file into a background thread to avoid building metadata in the main thread. This is not all that costly, but it is O(n) where n is the number of arrays in the tree, so it can start to add up for trees with a lot of parameters. Jul 12, 2024
@copybara-service copybara-service bot force-pushed the test_651805233 branch 5 times, most recently from 9758a16 to aa62c9a Compare July 18, 2024 15:34
PiperOrigin-RevId: 653631904
@copybara-service copybara-service bot merged commit 097a23b into main Jul 18, 2024
@copybara-service copybara-service bot deleted the test_651805233 branch July 18, 2024 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