Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Yuri Mizushima
+1 (non-binding) I checked * shasum and signature * pub/sub and Java Functions * connectors * stateful functions Thank you for your work! -- Yuri Mizushima yumiz...@yahoo-corp.jp "PengHui Li" wrote: Sorry for the typo, ``` 9f7304c3d35db97998a16a46362433c43c61d20eb3d2f51eb431

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Guangning E
+1(non-binding) * Build bin package and image * pub/sub * Run as a cluster Thanks, Guangning Yuri Mizushima 于2021年6月9日周三 下午5:09写道: > +1 (non-binding) > > I checked > * shasum and signature > * pub/sub and Java Functions > * connectors > * stateful functions > > Thank you for your work! > > --

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Enrico Olivelli
PengHui, I am testing the release and at the moment everything is good. But I found this problem while trying to upgrade a project that uses Transactions (Pulsar JMS client), it may be a showstopper for the adoption of Transactions. https://github.com/apache/pulsar/issues/10871 The problem is ab

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Hang Chen
Hi Penghui, I have do the following testing, * build bin package form source code * deploy on our test cluster and run pub-sub with pulsar-perf * deploy KoP on Pulsar cluster and run pub-sub with kafka-perf all works well. However, when i import `pulsar-common` and `pulsar-

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Michael Marshall
> But I found this problem while trying to upgrade a project that uses > Transactions (Pulsar JMS client), it may be a showstopper for the > adoption of Transactions. > https://github.com/apache/pulsar/issues/10871 I put together a PR that will allow the transaction topic to get created regardless

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Matteo Merli
Since transactions is a new feature and auto-creation of topics is by default enabled, I'd be more inclined to let the current RC go through. There is an easy workaround for this issue if someone wants to use transactions with no auto topic creation, which is to manually create these topics. We ca

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Andrey Yegorov
We talked about having pulsar-adapters released at the same time as 2.8 / being a part of the release process. https://github.com/apache/pulsar-adapters I don't see artifacts from the pulsar-adapters among the staged maven artifacts. I.e. pulsar-log4j2-appender is not there. pulsar-adapters were m

[GitHub] [pulsar-helm-chart] michaeljmarshall commented on pull request #123: WIP: Prevent breaking changes when using non-root container image for 2.9.0

2021-06-09 Thread GitBox
michaeljmarshall commented on pull request #123: URL: https://github.com/apache/pulsar-helm-chart/pull/123#issuecomment-858160029 This is now targeting the 2.9.0 release. I am working on a doc that will help coordinate all of the work required to move pulsar to default to using non-root co

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Chris Bartholomew
If we are going to let the release go out with a known regression, I think we need to clearly document it in the release notes so that folks can make a proper decision about whether or not they want to upgrade to this release. Chris On Wednesday, June 9, 2021, 05:42:13 PM EDT, Matteo Merli

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Jerry Peng
Hi all, I noticed another issue with the current release candidate. The java-instance.jar that is used as the root classloader for Pulsar Functions still contains too many dependencies. It should only contain the following deps: 1. pulsar-io-core 2. pulsar-functions-api 3. pulsar-cli

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Matteo Merli
On Wed, Jun 9, 2021 at 5:39 PM Chris Bartholomew wrote: > > If we are going to let the release go out with a known regression, I think > we need to clearly document it in the release notes so that folks can make a > proper decision about whether or not they want to upgrade to this release. I t

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Matteo Merli
I agree with Jerry on this one, since it impairs the classpath for existing functions. -- Matteo Merli On Wed, Jun 9, 2021 at 5:40 PM Jerry Peng wrote: > > Hi all, > > I noticed another issue with the current release candidate. The > java-instance.jar that is used as the root classloader for

[GitHub] [pulsar-helm-chart] jiahaoc1993 opened a new issue #125: Append Configuration of ImagePullSecret

2021-06-09 Thread GitBox
jiahaoc1993 opened a new issue #125: URL: https://github.com/apache/pulsar-helm-chart/issues/125 **Is your feature request related to a problem? Please describe.** Currently it is difficult to pull dockerhub images if not logged in. **Describe the solution you'd like** In chart'

[GitHub] [pulsar-helm-chart] jiahaoc1993 opened a new pull request #126: Feat/pullimagesecret support

2021-06-09 Thread GitBox
jiahaoc1993 opened a new pull request #126: URL: https://github.com/apache/pulsar-helm-chart/pull/126 Fixes #125 ### Motivation support the image pull secret -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [pulsar-client-node] k2la commented on issue #161: Looks like the pulsar-client-node can not run with the current pulsar master code

2021-06-09 Thread GitBox
k2la commented on issue #161: URL: https://github.com/apache/pulsar-client-node/issues/161#issuecomment-858331382 From your log, it looks like pulsar-client-cpp is not installed. Please try again after installing pulsar-client-cpp. Also, please tell us the system configuration (OS

Re: [VOTE] Pulsar Release 2.8.0 Candidate 2

2021-06-09 Thread Enrico Olivelli
Jerry, Matteo Il giorno gio 10 giu 2021 alle ore 02:52 Matteo Merli ha scritto: > > I agree with Jerry on this one, since it impairs the classpath for > existing functions. I strongly believe that #10878 is to be discussed more, see my comments here https://github.com/apache/pulsar/pull/10878#pu