Skip to content

feat(go worker): copy over a bit more logic from python#5312

Draft
michaelkedar wants to merge 3 commits intogoogle:masterfrom
michaelkedar:👷🙅🐍🪛

Hidden character warning

The head ref may contain hidden characters: "\ud83d\udc77\ud83d\ude45\ud83d\udc0d\ud83e\ude9b"
Draft

feat(go worker): copy over a bit more logic from python#5312
michaelkedar wants to merge 3 commits intogoogle:masterfrom
michaelkedar:👷🙅🐍🪛

Conversation

@michaelkedar
Copy link
Copy Markdown
Member

  • Clear the ImportFindings in Datastore when import is successful
  • send a gcs_retry pub/sub message on (what will be) the failed-tasks topic when writing to GCS fails after writing datastore entities
  • log the task latency that we have a log-based metric set up for

Might have merge conflicts with #5310, so this can be merged afterwards.

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