modern-data-warehouse-dataops icon indicating copy to clipboard operation
modern-data-warehouse-dataops copied to clipboard

Add Unit tests and monitoring capability

Open sreedhar-guda opened this issue 1 year ago • 2 comments

Type of PR

  • Documentation changes
  • Code changes
  • Test changes

Purpose

Implementation samples which include:

  • Unit testcases which can be made part of CI process
  • Monitoring and telemetry using OpenTelemetry

Author pre-publish checklist

  • [x] Added test to prove my fix is effective or new feature works
  • [x] No PII in logs
  • [x] Made corresponding changes to the documentation

Issues Closed or Referenced

  • Closes #issue_number
  • References #issue_number

sreedhar-guda avatar Aug 27 '24 12:08 sreedhar-guda

Feedback from Elena and Lace as of Aug 27:

  • Make the main notebook the good one (move the unit tcs else where). --> TO DO
  • It seems a lot, could make it harder for the users to follow.
  • Good to separate Fabric specific concepts from others (like unit testing and OTEL) --> TO DO, we already have these files as separate ones. Need to find a good place to move them.

sreedhar-guda avatar Aug 28 '24 18:08 sreedhar-guda

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Monitoring write-up is now part of #867

sreedhar-guda avatar Nov 22 '24 14:11 sreedhar-guda