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

refactor(xmlupload): _upload_stashed_xml_texts (DEV-2710) #532

Merged

Conversation

Nora-Olivia-Ammann
Copy link
Collaborator

No description provided.

@linear
Copy link

linear bot commented Sep 27, 2023

DEV-2710 refactor _upload_stashed_xml_texts

Diagnostics showed, that if the stashed links are large enough, there is a significant gap between one update of a stashed XML text till the next. The problem does not lie with the dsp-api, but with the Python code.

In order to diagnose what aspect of the stash causes the delay, the function "_upload_stashed_xml_texts" in xml_upload.py will be refactored. After that, the analysis will be run again.

@Nora-Olivia-Ammann Nora-Olivia-Ammann self-assigned this Sep 27, 2023
@Nora-Olivia-Ammann Nora-Olivia-Ammann changed the title refactor(xmlupload.py): _upload_stashed_xml_texts (DEV-2710) refactor(xmlupload): _upload_stashed_xml_texts (DEV-2710) Sep 28, 2023
Co-authored-by: Johannes Nussbaum <39048939+jnussbaum@users.noreply.github.com>
Co-authored-by: Johannes Nussbaum <39048939+jnussbaum@users.noreply.github.com>
Co-authored-by: Johannes Nussbaum <39048939+jnussbaum@users.noreply.github.com>
Co-authored-by: Johannes Nussbaum <39048939+jnussbaum@users.noreply.github.com>
Co-authored-by: Johannes Nussbaum <39048939+jnussbaum@users.noreply.github.com>
@Nora-Olivia-Ammann Nora-Olivia-Ammann merged commit 9d2dd35 into main Sep 28, 2023
4 checks passed
@Nora-Olivia-Ammann Nora-Olivia-Ammann deleted the wip/dev-2710-refactor-_upload_stashed_xml_texts branch September 28, 2023 12:25
@daschbot daschbot mentioned this pull request Sep 28, 2023
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.

None yet

2 participants