Skip to content

feat: migrate to OpenTelemetry and fix MERGE when only key columns#121

Merged
devlead merged 1 commit intoWCOMAB:mainfrom
devlead:feature/noUpdateIfOnlyKeys
Mar 18, 2026
Merged

feat: migrate to OpenTelemetry and fix MERGE when only key columns#121
devlead merged 1 commit intoWCOMAB:mainfrom
devlead:feature/noUpdateIfOnlyKeys

Conversation

@devlead
Copy link
Copy Markdown
Member

@devlead devlead commented Mar 18, 2026

  • Replace Application Insights with OpenTelemetry
    • Use Azure.Monitor.OpenTelemetry.Exporter, Microsoft.Azure.Functions.Worker.OpenTelemetry, OpenTelemetry.Extensions.Hosting, OpenTelemetry.Exporter.Console
    • Set host.json telemetryMode to OpenTelemetry and remove applicationInsights config
  • Configure worker logging: AddSimpleConsole, Logging from config, filter SqlBulkSyncFunction at Information
  • host.json: set default, Host.Results, Function log level to Information
  • README: add Logging__LogLevel__Default to example local.settings.json
  • SqlStatementExtensions: omit WHEN MATCHED THEN UPDATE when there are no updatable columns (only primary/identity), avoid empty SET and format/argument reorder
  • Bump Microsoft.Data.SqlClient to 7.0.0

- Replace Application Insights with OpenTelemetry
  - Use Azure.Monitor.OpenTelemetry.Exporter, Microsoft.Azure.Functions.Worker.OpenTelemetry,
    OpenTelemetry.Extensions.Hosting, OpenTelemetry.Exporter.Console
  - Set host.json telemetryMode to OpenTelemetry and remove applicationInsights config
- Configure worker logging: AddSimpleConsole, Logging from config, filter SqlBulkSyncFunction at Information
- host.json: set default, Host.Results, Function log level to Information
- README: add Logging__LogLevel__Default to example local.settings.json
- SqlStatementExtensions: omit WHEN MATCHED THEN UPDATE when there are no updatable columns
  (only primary/identity), avoid empty SET and format/argument reorder
- Bump Microsoft.Data.SqlClient to 7.0.0
@devlead devlead merged commit 1c8e814 into WCOMAB:main Mar 18, 2026
2 checks passed
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