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

[WC-2742] refactor: improve export performance #1365

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

iobuhov
Copy link
Collaborator

@iobuhov iobuhov commented Dec 4, 2024

Pull request type

Refactoring (e.g. file rename, variable rename, etc.)


Description

Improve export performanc by splitting read in batch operations + calling "get" on all items during read.

What should be covered while testing?

Export of small and big table should be enough.

@iobuhov iobuhov requested a review from a team as a code owner December 4, 2024 13:10
@iobuhov iobuhov force-pushed the fix/wc-2742/export-performance branch from 3ecc723 to f9755c5 Compare December 10, 2024 09:44
@iobuhov iobuhov changed the title [Draft] refactor: improve export performance [WC-2742] refactor: improve export performance Dec 10, 2024
@gjulivan
Copy link
Collaborator

changelog and data widget version bump 2.28.3

@gjulivan
Copy link
Collaborator

add changelog and bump version

@iobuhov iobuhov force-pushed the fix/wc-2742/export-performance branch 2 times, most recently from 32fc144 to 7a69ef6 Compare December 20, 2024 11:29
@iobuhov iobuhov force-pushed the fix/wc-2742/export-performance branch from 7a69ef6 to be1f62f Compare January 7, 2025 09:03
gjulivan
gjulivan previously approved these changes Jan 15, 2025
leonardomendix
leonardomendix previously approved these changes Jan 16, 2025
@iobuhov iobuhov dismissed stale reviews from leonardomendix and gjulivan via 17f5511 January 16, 2025 15:50
@iobuhov iobuhov force-pushed the fix/wc-2742/export-performance branch from be1f62f to 17f5511 Compare January 16, 2025 15:50
@iobuhov iobuhov force-pushed the fix/wc-2742/export-performance branch from 17f5511 to d05b24f Compare January 30, 2025 12:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants