You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/05/12 19:08:12 UTC
[GitHub] [pulsar] flowchartsman opened a new issue #10560: [Go Functions] support state storage API
flowchartsman opened a new issue #10560:
URL: https://github.com/apache/pulsar/issues/10560
Java and Python pulsar functions have access to the state API. This is very useful, so it's important for the Go SDK to support it too, for feature parity.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] flowchartsman closed issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
flowchartsman closed issue #10560:
URL: https://github.com/apache/pulsar/issues/10560
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] liangyuanpeng commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
liangyuanpeng commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-903045313
Ran had create the first PR for bk-client-go (add proto file). see https://github.com/apache/bookkeeper/pull/2750
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] liangyuanpeng edited a comment on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
liangyuanpeng edited a comment on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-903045313
Ran created the first PR for bk-client-go (add proto file). see https://github.com/apache/bookkeeper/pull/2750
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] flowchartsman commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
flowchartsman commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-908378608
If we're going to have `bk-client-go`, I probably don't need to file an issue for pulsar-client-go then, unless there are broader non-function-SDK uses for bk integration into the go client.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] flowchartsman commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
flowchartsman commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-908378608
If we're going to have `bk-client-go`, I probably don't need to file an issue for pulsar-client-go then, unless there are broader non-function-SDK uses for bk integration into the go client.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] flowchartsman edited a comment on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
flowchartsman edited a comment on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-939154033
Edit: sorry got this mixed up with another issue. Ignore.
On Fri, Oct 8, 2021 at 5:52 PM G ***@***.***> wrote:
> What is the status of this?
>
> The PR created by @wolfstudy <https://github.com/wolfstudy> lacks just
> the README - apache/bookkeeper#2750
> <https://github.com/apache/bookkeeper/pull/2750> Can we proceed by
> providing it, and moving forward?
>
> --
>
> Marking this as duplicate of #5138
> <https://github.com/apache/pulsar/issues/5138>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/pulsar/issues/10560#issuecomment-939135546>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAHCJJE2XQAB3YPYREA2N6LUF5RZHANCNFSM44ZF7OEQ>
> .
> Triage notifications on the go with GitHub Mobile for iOS
> <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
> or Android
> <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
>
>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] flowchartsman commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
flowchartsman commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-939154033
Only thing missing is basically a couple of tests. I can knock it out this
weekend. The only difficulty is that there’s not really a harness in place
for testing them meaningfully, so I’ve been putting it off and running my
own project out of my fork. But I’ll get on it.
On Fri, Oct 8, 2021 at 5:52 PM G ***@***.***> wrote:
> What is the status of this?
>
> The PR created by @wolfstudy <https://github.com/wolfstudy> lacks just
> the README - apache/bookkeeper#2750
> <https://github.com/apache/bookkeeper/pull/2750> Can we proceed by
> providing it, and moving forward?
>
> --
>
> Marking this as duplicate of #5138
> <https://github.com/apache/pulsar/issues/5138>
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/pulsar/issues/10560#issuecomment-939135546>,
> or unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAHCJJE2XQAB3YPYREA2N6LUF5RZHANCNFSM44ZF7OEQ>
> .
> Triage notifications on the go with GitHub Mobile for iOS
> <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
> or Android
> <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
>
>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] wolfstudy commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
wolfstudy commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-845612066
So when it was supported by Go Function, we temporarily ignored this function.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] genert commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
genert commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-939135546
What is the status of this?
The PR created by @wolfstudy lacks just the README - https://github.com/apache/bookkeeper/pull/2750 Can we proceed by providing it, and moving forward?
--
Marking this as duplicate of https://github.com/apache/pulsar/issues/5138
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] wolfstudy commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
wolfstudy commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-845611770
Hello @flowchartsman, as you said, we need to support state-related functions in Go Function, but this seems to be more difficult because we first need to support the Go client in BookKeeper to allow Go Function to access.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] flowchartsman commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
flowchartsman commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-846050108
So you're saying this needs a dependent issue for pulsar-client-go to support bookeeper interaction? If so, I can file one.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] genert edited a comment on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
genert edited a comment on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-939135546
This is feature that holds us back from using Golang for Pulsar Functions. What is the status of this?
The PR created by @wolfstudy lacks just the README - https://github.com/apache/bookkeeper/pull/2750
Can we proceed by providing it, and moving forward? If none of you have time for this, please let me know, and I will take over from here, and do these by myself.
--
Marking this as duplicate of https://github.com/apache/pulsar/issues/5138
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] liangyuanpeng commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
liangyuanpeng commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-902807871
> So you're saying this needs a dependent issue for pulsar-client-go to support bookeeper interaction? If so, I can file one.
I think it's true.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] flowchartsman commented on issue #10560: [Go Functions] support state storage API
Posted by GitBox <gi...@apache.org>.
flowchartsman commented on issue #10560:
URL: https://github.com/apache/pulsar/issues/10560#issuecomment-951091551
Duplicate of #5138
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org