Skip to content

Conversation

@azure-sdk
Copy link
Collaborator

@azure-sdk azure-sdk commented Dec 22, 2025

Breaking changes are due to service Swagger fixes.
https://dev.azure.com/azure-sdk/public/_build/results?buildId=5692401&view=logs&jobId=b70e5e73-bbb6-5567-0939-8415943fadb9&j=b70e5e73-bbb6-5567-0939-8415943fadb9&t=57f096e1-36e2-50f7-6e90-7a231ead9f90

[Automation] Generate Fluent Lite from Swagger specification/operationalinsights/resource-manager/Microsoft.OperationalInsights/OperationalInsights/readme.md#package-2025-07-01

…nalinsights/resource-manager/Microsoft.OperationalInsights/OperationalInsights/readme.md#package-2025-07-01
@github-actions github-actions bot added the Mgmt This issue is related to a management-plane library. label Dec 22, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 22, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

com.azure.resourcemanager:azure-resourcemanager-loganalytics

@XiaofeiCao
Copy link
Contributor

/azp run java - loganalytics - mgmt - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@XiaofeiCao XiaofeiCao marked this pull request as ready for review December 22, 2025 05:46
Copilot AI review requested due to automatic review settings December 22, 2025 05:46
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the Azure SDK for Java's Log Analytics resource manager library by regenerating code from the updated Swagger specification (package-2025-07-01). The changes primarily involve refactoring client implementation patterns, updating API versions, and introducing new features.

Key Changes

  • Refactored synchronous and asynchronous method implementations across multiple client classes
  • Added new SummaryLogsOperations functionality with full CRUD operations
  • Updated import statements to use com.azure.core.management.SystemData instead of custom types
  • Added new workspace properties (replication and failover)

Reviewed changes

Copilot reviewed 52 out of 342 changed files in this pull request and generated no comments.

Show a summary per file
File Description
WorkspacePurgesClientImpl.java Refactored sync methods, updated service interface name, added ClientLogger, replaced hardcoded API versions
WorkspaceImpl.java Updated SystemData import, changed date types to OffsetDateTime, added replication/failover properties
UsagesClientImpl.java Refactored pagination methods, updated service interface name, added sync implementations
TablesClientImpl.java Refactored LRO operations to use sync methods, updated service interface name, added BinaryData support
TableImpl.java Updated SystemData import
SummaryLogsOperationsImpl.java New file implementing summary logs operations
SummaryLogsOperationsClientImpl.java New file with full client implementation for summary logs
SummaryLogsImpl.java New file with model implementation
StorageInsightConfigsClientImpl.java Refactored sync methods, updated service interface name, added ClientLogger
SharedKeysOperationsClientImpl.java Refactored sync methods, updated service interface name
SchemasClientImpl.java Refactored sync methods, updated service interface name
SavedSearchesClientImpl.java Refactored sync methods, updated service interface name
QueryPacksClientImpl.java Refactored pagination and sync methods, updated service interface name
OperationsClientImpl.java Refactored pagination methods, updated service interface name
NetworkSecurityPerimeterConfigurationImpl.java New file for NSP configuration model

Copy link
Member

@haolingdong-msft haolingdong-msft left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly looks good to me, add one comment on api-specs.yaml

@XiaofeiCao XiaofeiCao merged commit 491f85f into Azure:main Dec 22, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Mgmt - Track 2 Mgmt This issue is related to a management-plane library.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants