-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
notification: add Go CDK pubsub support #895
Conversation
Add the gocdk_pub_sub package, which supports the Go Cloud Development Kit pubsub API. Link in all current providers. Update the notification scaffold.
Nice! There are some build errors: ''' |
FYI: the build log |
Can this gocloud.dev/pubsub be moved/cloned to github? Seems this is not new: googleapis/google-cloud-go#820 happened more than one year ago. |
I'm working on this; it is a problem with go-import redirects for the gocloud.dev import paths. |
Tests are passing now. |
Thanks a lot! Is there any way to avoid this in the future? These go import redirects seem fragile. |
We're automating our process so this won't happen again. More generally, the Go ecosystem will soon have proxies, which will pretty much end availability issues. Run |
@jba SeaweedFS use the notification to replicate files across systems. The current PR implemented SendMessage(). We need another one to ReceiveMessage(), similar to https://github.com/chrislusf/seaweedfs/blob/master/weed/replication/sub/notification_google_pub_sub.go#L94 Could you please also help here? Thanks! |
Sure, I'll try to get to it this week. |
Add the gocdk_pub_sub package, which supports the Go Cloud Development
Kit pubsub API.
Link in all current providers.
Update the notification scaffold.