Jenkins build is back to stable : pulsar-master #467

2018-05-13 Thread Apache Jenkins Server
See 




[CANCELED] [VOTE] Pulsar Release 2.0.0-rc1-incubating Candidate 3

2018-05-13 Thread Matteo Merli
Canceling this vote thread to remove pre-packaged jar used in unit tests.

Matteo

On Sat, May 12, 2018 at 7:18 PM Dave Fisher  wrote:

> Hi Pulsar Devs,
>
> Thanks Justin! I know that Pulsar Functions are new.
>
> $ jar tvf
> apache-pulsar-2.0.0-rc1-incubating/pulsar-functions/utils/src/test/resources/multifunction.jar
>  0 Wed Dec 13 01:00:28 PST 2017 META-INF/
>147 Wed Dec 13 01:00:28 PST 2017 META-INF/MANIFEST.MF
>  0 Wed Dec 13 00:58:36 PST 2017 org/
>  0 Wed Dec 13 00:58:36 PST 2017 org/apache/
>  0 Wed Dec 13 00:58:36 PST 2017 org/apache/pulsar/
>  0 Wed Dec 13 00:58:36 PST 2017 org/apache/pulsar/functions/
>  0 Wed Dec 13 00:58:36 PST 2017 org/apache/pulsar/functions/runtime/
>  0 Wed Dec 13 00:58:56 PST 2017
> org/apache/pulsar/functions/runtime/functioncache/
>683 Wed Dec 13 00:57:10 PST 2017
> org/apache/pulsar/functions/runtime/functioncache/AddFunction.class
>
> AddFunction is nowhere in the source. This is something that must be
> corrected.
>
> Sorry, but I am going to change my votes to +0 until there is an
> explanation. One option would be to remove this test …
>
> Please keep the eye out for jars and tars in the source.
>
> Regards,
> Dave
>
> On May 12, 2018, at 3:52 PM, Justin Mclean 
> wrote:
>
> Hi,
>
> Sorry but it’s -1 binding due to compiled code in source release.
>
> Frankly I’m surprised to see so many +1s with this obvious issue.
>
> I checked:
> - incubating in name
> - signatures and hashes correct
> - DISCLAIMER exists
> - LICENSE and NOTICE good
> - Unexpected jar in source release [1]
> - All source files have headers
>
> Thanks,
> Justin
>
>
> 1.  pulsar-functions/utils/src/test/resources/multifunction.jar
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>
> --
Matteo Merli



Re: Planning for Pulsar 2.1 release

2018-05-13 Thread Jerry Peng
+1

On Sun, May 13, 2018 at 6:35 PM Jia Zhai  wrote:

>  👍
>
> On Fri, May 11, 2018 at 1:51 AM, Sanjeev Kulkarni 
> wrote:
>
> > Looks good!
> >
> > On Thu, May 10, 2018 at 9:12 AM, Matteo Merli  wrote:
> >
> > > Now that 2.0 is on the way, I'd like to get some items that we're
> > targeting
> > > to get into a 2.1 release:
> > >
> > >  * Connector framework (pulsar-io), built on top of Pulsar Functions
> > >  * Go client library (based on C++ Pulsar client lib)
> > >  * Tooling and docs around rack-aware deployment
> > >  * Tiered storage framework (with S3 as first supported backend)
> > >  * Schema integration for unctions
> > >  * Distributed state in functions
> > >  * Avro schema support
> > >  * Improvements in pulsar-admin cli tool
> > >
> > > The idea would be to have a 2.1 release in early June.
> > >
> > > What are the thoughts around this?
> > >
> > > Matteo
> > > --
> > > Matteo Merli
> > > 
> > >
> >
>


Re: Planning for Pulsar 2.1 release

2018-05-13 Thread Jia Zhai
 👍

On Fri, May 11, 2018 at 1:51 AM, Sanjeev Kulkarni 
wrote:

> Looks good!
>
> On Thu, May 10, 2018 at 9:12 AM, Matteo Merli  wrote:
>
> > Now that 2.0 is on the way, I'd like to get some items that we're
> targeting
> > to get into a 2.1 release:
> >
> >  * Connector framework (pulsar-io), built on top of Pulsar Functions
> >  * Go client library (based on C++ Pulsar client lib)
> >  * Tooling and docs around rack-aware deployment
> >  * Tiered storage framework (with S3 as first supported backend)
> >  * Schema integration for unctions
> >  * Distributed state in functions
> >  * Avro schema support
> >  * Improvements in pulsar-admin cli tool
> >
> > The idea would be to have a 2.1 release in early June.
> >
> > What are the thoughts around this?
> >
> > Matteo
> > --
> > Matteo Merli
> > 
> >
>


Jenkins build is unstable: pulsar-master #466

2018-05-13 Thread Apache Jenkins Server
See 




Re: [VOTE] Pulsar Release 2.0.0-rc1-incubating Candidate 3

2018-05-13 Thread Matteo Merli
Hi Matt,

can you expand a bit more on the issue, because I'm not 100% sure I
understand it correctly.

> -1 as well. For strange build requirements that need to inject class
files,
that would still only be applied to the binary artifacts

The code in question is testing a feature that takes Jars from a user and
execute the code contained. Indeed the Jar should not be in the sources,
though I believe the test is correct here: we need to validate the specific
feature by dynamically loading a Jar.

> (e.g., injecting a
module-info.class file at build time rather than requiring Java 9+ to build
regardless).

I don't see any place in the code where we are using "module-info.class"
and the project requires Java 8 and not 9.

Thanks,
Matteo

On Sun, May 13, 2018 at 8:46 AM Matt Sicker  wrote:

> -1 as well. For strange build requirements that need to inject class files,
> that would still only be applied to the binary artifacts (e.g., injecting a
> module-info.class file at build time rather than requiring Java 9+ to build
> regardless).
>
> On 12 May 2018 at 21:17, Dave Fisher  wrote:
>
> > Hi Pulsar Devs,
> >
> > Thanks Justin! I know that Pulsar Functions are new.
> >
> > $ jar tvf apache-pulsar-2.0.0-rc1-incubating/pulsar-functions/
> > utils/src/test/resources/multifunction.jar
> >  0 Wed Dec 13 01:00:28 PST 2017 META-INF/
> >147 Wed Dec 13 01:00:28 PST 2017 META-INF/MANIFEST.MF
> >  0 Wed Dec 13 00:58:36 PST 2017 org/
> >  0 Wed Dec 13 00:58:36 PST 2017 org/apache/
> >  0 Wed Dec 13 00:58:36 PST 2017 org/apache/pulsar/
> >  0 Wed Dec 13 00:58:36 PST 2017 org/apache/pulsar/functions/
> >  0 Wed Dec 13 00:58:36 PST 2017 org/apache/pulsar/functions/runtime/
> >  0 Wed Dec 13 00:58:56 PST 2017 org/apache/pulsar/functions/
> > runtime/functioncache/
> >683 Wed Dec 13 00:57:10 PST 2017 org/apache/pulsar/functions/
> > runtime/functioncache/AddFunction.class
> >
> > AddFunction is nowhere in the source. This is something that must be
> > corrected.
> >
> > Sorry, but I am going to change my votes to +0 until there is an
> > explanation. One option would be to remove this test …
> >
> > Please keep the eye out for jars and tars in the source.
> >
> > Regards,
> > Dave
> >
> > On May 12, 2018, at 3:52 PM, Justin Mclean 
> > wrote:
> >
> > Hi,
> >
> > Sorry but it’s -1 binding due to compiled code in source release.
> >
> > Frankly I’m surprised to see so many +1s with this obvious issue.
> >
> > I checked:
> > - incubating in name
> > - signatures and hashes correct
> > - DISCLAIMER exists
> > - LICENSE and NOTICE good
> > - Unexpected jar in source release [1]
> > - All source files have headers
> >
> > Thanks,
> > Justin
> >
> >
> > 1.  pulsar-functions/utils/src/test/resources/multifunction.jar
> > -
> > To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> > For additional commands, e-mail: general-h...@incubator.apache.org
> >
> >
> >
>
>
> --
> Matt Sicker 
>
-- 
Matteo Merli



Build failed in Jenkins: pulsar-master #465

2018-05-13 Thread Apache Jenkins Server
See 


Changes:

[guosijie] Seperate cmd line connector interface to explicit source and sink

[guosijie] Allow padding at end of last BlockAwareSegmentInputStream (#1739)

--
[...truncated 458.66 KB...]
[INFO] 
[INFO] 
[INFO] --- maven-clean-plugin:2.4.1:clean (default-clean) @ pulsar-client-admin 
---
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- license-maven-plugin:3.0.rc1:check (default-cli) @ 
pulsar-client-admin ---
[INFO] Checking licenses...
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-dependency-plugin:2.10:build-classpath (build-classpath) @ 
pulsar-client-admin ---
[INFO] Wrote classpath file 
'
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.5:process (process-resource-bundles) 
@ pulsar-client-admin ---
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-resources-plugin:2.7:resources (default-resources) @ 
pulsar-client-admin ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-compiler-plugin:3.7.0:compile (default-compile) @ 
pulsar-client-admin ---
[INFO] No sources to compile
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-resources-plugin:2.7:testResources (default-testResources) @ 
pulsar-client-admin ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-compiler-plugin:3.7.0:testCompile (default-testCompile) @ 
pulsar-client-admin ---
[INFO] No sources to compile
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-surefire-plugin:2.21.0:test (default-test) @ 
pulsar-client-admin ---
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-dependency-plugin:2.10:build-classpath (build-classpath) @ 
pulsar-client-admin ---
[INFO] Skipped writing classpath file 
'
  No changes found.
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-remote-resources-plugin:1.5:process (process-resource-bundles) 
@ pulsar-client-admin ---
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-resources-plugin:2.7:resources (default-resources) @ 
pulsar-client-admin ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 

[INFO] Copying 3 resources
[INFO] Copying 3 resources
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-compiler-plugin:3.7.0:compile (default-compile) @ 
pulsar-client-admin ---
[INFO] No sources to compile
[WARNING] Failed to notify spy hudson.maven.Maven3Builder$JenkinsEventSpy: 
java.util.concurrent.ExecutionException: Invalid object ID 13 iota=75
[INFO] 
[INFO] --- maven-resources-plugin:2.7:testResources (default-testResources) @ 
pulsar-client-admin ---
[INFO] Using 'UTF-8' encoding to copy filtered resources.
[INFO] skip non existing resourceDirectory 


Re: [VOTE] Pulsar Release 2.0.0-rc1-incubating Candidate 3

2018-05-13 Thread Matt Sicker
-1 as well. For strange build requirements that need to inject class files,
that would still only be applied to the binary artifacts (e.g., injecting a
module-info.class file at build time rather than requiring Java 9+ to build
regardless).

On 12 May 2018 at 21:17, Dave Fisher  wrote:

> Hi Pulsar Devs,
>
> Thanks Justin! I know that Pulsar Functions are new.
>
> $ jar tvf apache-pulsar-2.0.0-rc1-incubating/pulsar-functions/
> utils/src/test/resources/multifunction.jar
>  0 Wed Dec 13 01:00:28 PST 2017 META-INF/
>147 Wed Dec 13 01:00:28 PST 2017 META-INF/MANIFEST.MF
>  0 Wed Dec 13 00:58:36 PST 2017 org/
>  0 Wed Dec 13 00:58:36 PST 2017 org/apache/
>  0 Wed Dec 13 00:58:36 PST 2017 org/apache/pulsar/
>  0 Wed Dec 13 00:58:36 PST 2017 org/apache/pulsar/functions/
>  0 Wed Dec 13 00:58:36 PST 2017 org/apache/pulsar/functions/runtime/
>  0 Wed Dec 13 00:58:56 PST 2017 org/apache/pulsar/functions/
> runtime/functioncache/
>683 Wed Dec 13 00:57:10 PST 2017 org/apache/pulsar/functions/
> runtime/functioncache/AddFunction.class
>
> AddFunction is nowhere in the source. This is something that must be
> corrected.
>
> Sorry, but I am going to change my votes to +0 until there is an
> explanation. One option would be to remove this test …
>
> Please keep the eye out for jars and tars in the source.
>
> Regards,
> Dave
>
> On May 12, 2018, at 3:52 PM, Justin Mclean 
> wrote:
>
> Hi,
>
> Sorry but it’s -1 binding due to compiled code in source release.
>
> Frankly I’m surprised to see so many +1s with this obvious issue.
>
> I checked:
> - incubating in name
> - signatures and hashes correct
> - DISCLAIMER exists
> - LICENSE and NOTICE good
> - Unexpected jar in source release [1]
> - All source files have headers
>
> Thanks,
> Justin
>
>
> 1.  pulsar-functions/utils/src/test/resources/multifunction.jar
> -
> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org
> For additional commands, e-mail: general-h...@incubator.apache.org
>
>
>


-- 
Matt Sicker