Skip to content

Conversation

Charles-Gagnon
Copy link
Contributor

  1. Add editorconfig with same stuff we ignore in test projects since we don't need to be as strict here
  2. Add MemoryDiagnoser to trigger test
  3. Move common stuff to base class - I'll be adding more trigger perf tests here soon which will reuse those
  4. Disable/Re-enable change tracking on the table after/before each test so that we start with a clean slate each time
  5. Delete Leases table after each run, otherwise we run into an issue where the functions are killed before they can release their leases and so end up skipping over the rows since the IDs are reused across multiple runs. This is kind of messy and would be worth revisiting in the future
  6. Adding logging statement to tests when they start waiting for product changes to come through, this can help figure out timeout issues since it tells you when it's actually started watching so you can see what it was doing before it timed out
  7. Add README section on troubleshooting tests on enabling debug logging. Made Enable debug logging during test runs #438 for tracking enabling this for all test runs

@Charles-Gagnon Charles-Gagnon merged commit 2af6fc5 into triggerbindings Nov 2, 2022
@Charles-Gagnon Charles-Gagnon deleted the chgagnon/perfFixes branch November 2, 2022 23:02
PBBlox pushed a commit to PBBlox/azure-functions-sql-extension that referenced this pull request Apr 6, 2025
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.

2 participants