redash icon indicating copy to clipboard operation
redash copied to clipboard

[Feature] Add environment variable for query retry

Open surgachsurgach opened this issue 2 years ago • 1 comments

What type of PR is this?

  • [ ] Refactor
  • [x] Feature
  • [ ] Bug Fix
  • [ ] New Query Runner (Data Source)
  • [ ] New Alert Destination
  • [ ] Other

Description

How is this tested?

  • [ ] Unit tests (pytest, jest)
  • [ ] E2E Tests (Cypress)
  • [ ] Manually
  • [ ] N/A

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

surgachsurgach avatar Jul 11 '22 10:07 surgachsurgach

Thanks for this effort. Please complete the Description and How is this tested? segments of the pull request template. This way reviewers can have context on this change.

susodapop avatar Jul 11 '22 19:07 susodapop

Hi @surgachsurgach , thanks for your contribution! Would you mind describing the PR further + running some tests for it? We also need to rebase off of master and resolve merge conflicts to re-run the CI.

Additionally, would you mind updating the documentation here with the new env var behavior?

guidopetri avatar Jul 15 '23 13:07 guidopetri