Patch api for Partition service
This change addresses the issue #2 (closed). Patch API will be able to update existing partitions. This includes:
- Update existing properties (same key, changing 'sensitive' field or changing 'value' field)
- Add new properties
Does not include deletion of properties. To do that, we would have to delete and re-create the partition.
To add this API, we also need to change to Redis based cache as VM cache causes inconsistencies when number of instances are more than 1.
Will follow up with a core-common change to add api to the client library.
Merge request reports
Activity
added AWS Azure Common Code labels
assigned to @syang, @dcarao, and @danielscholl
assigned to @ethiraj
@dcarao @danielscholl @ethiraj could you please review? The trusted build is green
added 2 commits
added 6 commits
-
224bc85a...324b6623 - 5 commits from branch
master - 60bfd609 - rebase and resolve conflict
-
224bc85a...324b6623 - 5 commits from branch
added 13 commits
-
60bfd609...6657439f - 10 commits from branch
master - 18d87944 - Merge branch 'master' of...
- a2020e48 - Merge branch 'master' of...
- 4bc9f590 - refactor redis cache
Toggle commit list-
60bfd609...6657439f - 10 commits from branch
@Nieten @wsmatth Please review, pipeline is green and this feature was requested by @wsmatth and others.
For AWS it will return 'not implemented' for now, when your ready you can implement this method.
I will review and merge it today. Let me know if you have any questions/comments.
Edited by ethiraj krishnamanaidu
added 4 commits
-
4bc9f590...12ab92bc - 2 commits from branch
master - 7addf852 - Merge branch 'master' of...
- 661684c0 - rebase, remove unused property
-
4bc9f590...12ab92bc - 2 commits from branch
mentioned in commit c5e5a8f8
-
Approved and Merged. @ajoshi19 Thanks for updating API spec and documentation
added MRNew Feature label
changed milestone to %M1 - Release 0.1
