Skip to content

Comments

refactor(services/azdls): Refactor raw request send in writer and bac…#6109

Merged
Xuanwo merged 2 commits intoapache:mainfrom
jorgehermo9:azdls-refactor-core
Apr 27, 2025
Merged

refactor(services/azdls): Refactor raw request send in writer and bac…#6109
Xuanwo merged 2 commits intoapache:mainfrom
jorgehermo9:azdls-refactor-core

Conversation

@jorgehermo9
Copy link
Contributor

As similar PRs before, I noticed some inconsistencies in how the requests were send in backend and writer. Now azdls_create and azdls_update behaves similiar to other methods such as azdls_read

@jorgehermo9 jorgehermo9 marked this pull request as ready for review April 27, 2025 12:24
@jorgehermo9 jorgehermo9 requested a review from Xuanwo as a code owner April 27, 2025 12:24
@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" labels Apr 27, 2025
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thank you @jorgehermo9 for this change.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Apr 27, 2025
@Xuanwo Xuanwo merged commit 68c99b3 into apache:main Apr 27, 2025
86 checks passed
@jorgehermo9 jorgehermo9 deleted the azdls-refactor-core branch April 27, 2025 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/refactor The PR does a refactor on code or has a title that begins with "refactor" size:M This PR changes 30-99 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants