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