forked from apache/pulsar
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Issue 4175] [pulsar-function-go] Add Go Function heartbeat (and gRPC…
… service) for production usage (apache#6031) Partial fix required for apache#4175. Addresses issue documented here: grpc/grpc-go#3310 Part of ongoing work to add the gRPC service for heartbeat functionality and related support for running Go functions. ### Motivation Progress was blocked by the issue documented above. ### Modifications 1. Updated ./generate.sh script in pulsar-function-go module to use the grpc plugin. 2. Rebuilt Go gRPC files using the grpc plugin. This updated the proto version and was mostly an additive change that included adding the new registration methods required for creating the Go gRPC service. ### Verifying this change - Unable to test changes because Go tests were already failing. Currently investigating. - More testing will be done in subsequent commits because the Go gRPC functionality is not yet complete. (Should be low impact.)
- Loading branch information
Showing
20 changed files
with
1,084 additions
and
261 deletions.
There are no files selected for viewing
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,3 +13,5 @@ require ( | |
| gopkg.in/natefinch/lumberjack.v2 v2.0.0 | ||
| gopkg.in/yaml.v2 v2.2.2 // indirect | ||
| ) | ||
|
|
||
| go 1.13 | ||
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -55,3 +55,4 @@ disk: 0 | |
| # retryDetails config | ||
| maxMessageRetries: 0 | ||
| deadLetterTopic: "" | ||
| expectedHealthCheckInterval: 3 | ||
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
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -58,4 +58,3 @@ func main() { | |
| consumer.Ack(msg) | ||
| } | ||
| } | ||
|
|
||
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
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
Oops, something went wrong.