getProjectionStatus #268
getProjectionStatus #268
Conversation
|
@George-Payne, could you add a snippet for that method like, e.g. in the .NET client? https://github.com/EventStore/EventStore-Client-Dotnet/blob/master/samples/projection-management/Program.cs#L294 |
| UnknownError, | ||
| } from "@eventstore/db-client"; | ||
|
|
||
| describe("getProjectionStatistics", () => { |
There was a problem hiding this comment.
SUGGESTION: I think it'd be more consistent if we use the same semantics in test names as in the method name.
| describe("getProjectionStatistics", () => { | |
| describe("getProjectionStatus", () => { |
| options: baseOptions, | ||
| }); | ||
|
|
||
| const { projectionStatus } = await this.getProjectionStatistics( |
There was a problem hiding this comment.
SUGGESTION: Shouldn't we mark getProjectionStatistics and GetProjectionStatisticsOptions as obsolete/deprecated?
There was a problem hiding this comment.
They're still a supported operation: https://github.com/EventStore/architecture-and-planning/blob/main/rfcs/021%20-%20gRPC%20Projections.md#supported-operations
|
Some confusion around the RFC, will close this. see: https://github.com/EventStore/architecture-and-planning/pull/100 superseeded by: #269 |
Implement
getProjectionStatus.ref: #238