Patch function for cosmosdb added #115
Conversation
Atharva1723
requested review from
NipunaRanasinghe and
shafreenAnfar
as code owners
|
|
|
@NipunaRanasinghe Is this correct? |
native/src/main/java/io/ballerinax/cosmosdb/DataplaneClient.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Ayesh Almeida <77491511+ayeshLK@users.noreply.github.com>
|
Done |
| @display {label: "Partition Key"} string partitionKey, | ||
| @display {label: "Patch Operations"} PatchOperation[] operations, | ||
| @display {label: "Optional Header Parameters"} PatchRequestOptions? requestOptions = ()) | ||
| returns cosmosdb:DocumentResponse|error = @java:Method { |
There was a problem hiding this comment.
| returns cosmosdb:DocumentResponse|error = @java:Method { | |
| returns DocumentResponse|error = @java:Method { |
| @@ -0,0 +1,41 @@ | |||
| import ballerina/log; | |||
There was a problem hiding this comment.
Missing license header
| import ballerina/log; | |
| // Copyright (c) 2024 WSO2 LLC. (http://www.wso2.org). | |
| // | |
| // WSO2 LLC. licenses this file to you under the Apache License, | |
| // Version 2.0 (the "License"); you may not use this file except | |
| // in compliance with the License. | |
| // You may obtain a copy of the License at | |
| // | |
| // http://www.apache.org/licenses/LICENSE-2.0 | |
| // | |
| // Unless required by applicable law or agreed to in writing, | |
| // software distributed under the License is distributed on an | |
| // "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY | |
| // KIND, either express or implied. See the License for the | |
| // specific language governing permissions and limitations | |
| // under the License. | |
| import ballerina/log; |
|
@Atharva1723 we need to add at-least 1 positive and negative test cases for the |
|
As discussed with @anupama-pathirage, we will merge the feature in its current state into a feature development branch, as @Atharva1723 appears to have been inactive over the past few weeks. Once the remaining suggestions are addressed from our side, we will send a separate PR. @Atharva1723 please note that we have decided to grant you the full credit allocated for this issue (taking into account the percentage of completion and delays on our end). Feel free to submit the contribution form and claim your credits to access our swag store 🙂 |
NipunaRanasinghe
merged commit ffb428a
into
ballerina-platform:feat-patch-operation
Sorry was busy in office past two weeks ...Should i raise another pr for the test cases ? |
@Atharva1723 Thats okay and welcome back! :) Yes that would be great if you can send a PR for the test cases. Please send it to the https://github.com/ballerina-platform/module-ballerinax-azure-cosmosdb/tree/feat-patch-operation branch |
Description
This update implements a function to patch a document in Azure Cosmos DB using the Ballerina programming language. The function allows for modifying existing fields and adding new fields to the specified document.
Fixes ballerina-platform/ballerina-library#4972 (comment)
One line release note:
Type of change
Please delete options that are not relevant.
Checklist:
Security checks