Merge feature/entities to main #180
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…n. (#186) * add factories * implement grpc worker * update to latest core, remove obsolete files * Apply suggestions from code review Co-authored-by: Jacob Viau <javia@microsoft.com> * fix renaming of variables * address PR feedback * propagate changes from microsoft/durabletask-protobuf/pull/14 * revise representation of operation results * update/simplify protobuf utils * simplify StateShim and ContextShim * some minor simplifications * add missing failure details. * simplify operationActions list useage, by always creating a fresh list when resetting. * Apply suggestions from code review Co-authored-by: Jacob Viau <javia@microsoft.com> --------- Co-authored-by: Jacob Viau <javia@microsoft.com>
* Add gRPC entity runner * Address PR comments
* draft * implement client * simplify query * propagate changes from microsoft/durabletask-protobuf/pull/14 * Apply suggestions from code review Co-authored-by: Jacob Viau <javia@microsoft.com> * address PR feedback * add continuation token to CleanEntityStorage operation * fix/simplify protobuf format * minor simplifications * add optional continueUntilComplete argument to CleanEntityStorageAsync * update to latest proto --------- Co-authored-by: Jacob Viau <javia@microsoft.com>
* draft * missing file * update to match latest version of DT.Core * updates * update to latest * minor simplification. * address PR feedback and apply some minor simplifications. * address more PR feedback * latest proto
sebastianburckhardt
added
durable-entities
* remove errorContext, fix how operation result is read * add TaskEntityOperation parameter to InitializeState * add missing parameter checking for entity IDs that are passed in * pass entity parameters to orchestration executor
* pass entity parameters to orchestration executor * remove erroneous check * pass EntityMessageEvent as the original object * latest proto
…r for entity messages (#207)
* revise EntityInstanceId implementation and checking * address PR feedback: add switch to enable/disable entity support and associated checking * one more validation check for entity names * remove checks that were supposed to be removed but got re-added during merge mess * address PR feedback
* rename StartOrchestration to ScheduleNewOrchestration and add ScheduledStartTime parameter * specify different defaults for clean entity storage (by default, everything should be cleaned) * support queries that return stateless entities, and add more properties to entity metadata * make the page size an optional parameter (if absent, backend determines what page size is convenient) * address PR feedback * update to latest protobuf definition * rename includeStateless to includeTransient * update proto * update proto submodule to latest main branch
jviau
approved these changes
Oct 18, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As in title.