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

allow input path override #1536

Closed
wants to merge 4 commits into from

Conversation

jrodal98
Copy link
Member

@jrodal98 jrodal98 commented Sep 2, 2022

Summary:

What is this stack

  • Bolt has been used in canary for weeks now and we haven't seen reports of any issues
  • It's time to productionize
  • Early in the stack, I rework some logic to raise the quality bar a bit. At the end of the stack, I delete the legacy runner.

What

  • When resuming a run, update the input path on the instance

Why

  • This makes it so that advertisers who fail during input data validation can update their input file and continue the run (instead of having to wait for the instance to expire)

Differential Revision: D39229715

Differential Revision: D39227358

fbshipit-source-id: 2088630c0805153c2147f72de6db19647b019151
Differential Revision: D39227339

fbshipit-source-id: bcc18c2268493645d219a601708f746af8b159d5
Differential Revision: D39229515

fbshipit-source-id: b5262ae3a279dc0cfe772c09548c9d09ef8c3490
Summary:
## What is this stack

- Bolt has been used in canary for weeks now and we haven't seen reports of any issues
- It's time to productionize
- Early in the stack, I rework some logic to raise the quality bar a bit. At the end of the stack, I delete the legacy runner.

## What

- When resuming a run, update the input path on the instance

## Why

- This makes it so that advertisers who fail during input data validation can update their input file and continue the run (instead of having to wait for the instance to expire)

Differential Revision: D39229715

fbshipit-source-id: 9ae7dada74232964a3e0cf5253a52e22ffca662e
@facebook-github-bot facebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported labels Sep 2, 2022
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D39229715

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants