You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bahir.apache.org by Robert Metzger <rm...@apache.org> on 2017/07/11 08:29:51 UTC

Re: [VOTE] Apache Bahir 2.1.1 (RC2)

+1 (binding)

- the staging repo looks good
- src archive looks good (no binaries, no snapshot dependencies)
- mvn clean install -DskipTests (including rat license check) passed on src
contents




On Fri, Jun 30, 2017 at 4:03 PM, Luciano Resende <lu...@gmail.com>
wrote:

> We are still short one vote, any PMC volunteers to review the release ?
>
> On Sat, Jun 17, 2017 at 5:29 PM, Christian Kadner <ck...@apache.org>
> wrote:
>
> > *+1* (binding)
> >
> > The distribution files look good and install build runs successfully.
> >
> > *Thanks Luciano!*
> >
> >
> > - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
> >
> > A few suggestions for future releases:
> >
> >
> > *1)* We may want to enhance the release-build.sh script to update the
> >     *library dependencies* in the module *README.md* files the
> anticipated
> >     release version (not "2.2.0-SNAPSHOT" or "2.1.0-SNAPSHOT").
> >
> >     Currently none of the README.md files show versions that work, i.e.
> ...
> >
> >     ./sql-streaming-akka/README.md:
> >
> >     libraryDependencies += "org.apache.bahir" %%
> "spark-sql-streaming-akka"
> > % "2.2.0-SNAPSHOT"
> >
> >     $ bin/spark-shell --packages
> > org.apache.bahir:spark-sql-streaming-akka_2.11:2.2.0-SNAPSHOT
> >
> >     ./sql-streaming-mqtt/README.md:
> >
> >     libraryDependencies += "org.apache.bahir" %%
> "spark-sql-streaming-mqtt"
> > % "2.1.0-SNAPSHOT"
> >
> >     $ bin/spark-shell --packages
> > org.apache.bahir:spark-sql-streaming-mqtt_2.11:2.1.0-SNAPSHOT
> >
> >     ...
> >
> >
> > *2) *There are *new* *deprecation warnings* we may want to address:
> >
> >
> > [WARNING] CloudantStreaming.scala:67: method registerTempTable in class
> > Dataset is deprecated: Use createOrReplaceTempView(viewName) instead.
> > [WARNING]           changesDataFrame.registerTempTable("
> > airportcodemapping")
> > [WARNING]                            ^
> > [WARNING] PubsubStreamSuite.scala:106: postfix operator milliseconds
> should
> > be enabled
> > [WARNING]     eventually(timeout(10000 milliseconds), interval(100
> > milliseconds)) {
> > [WARNING]                              ^
> > [WARNING] PubsubStreamSuite.scala:106: postfix operator milliseconds
> should
> > be enabled
> > [WARNING]     eventually(timeout(10000 milliseconds), interval(100
> > milliseconds)) {
> > [WARNING]                                                          ^
> > [WARNING] PubsubStreamSuite.scala:130: postfix operator milliseconds
> should
> > be enabled
> > [WARNING]     eventually(timeout(10000 milliseconds), interval(100
> > milliseconds)) {
> > [WARNING]                              ^
> > [WARNING] PubsubStreamSuite.scala:130: postfix operator milliseconds
> should
> > be enabled
> > [WARNING]     eventually(timeout(10000 milliseconds), interval(100
> > milliseconds)) {
> > [WARNING]                                                          ^
> >
> >     ... and some *older* ones ...
> >
> > [WARNING] MQTTStreamSourceSuite.scala:57: constructor SQLContext in
> class
> > SQLContext is deprecated: Use SparkSession.builder instead
> > [WARNING]     val sqlContext: SQLContext = new SQLContext(sc)
> > [WARNING]                                  ^
> > [WARNING] MQTTStreamSourceSuite.scala:128: constructor SQLContext in
> class
> > SQLContext is deprecated: Use SparkSession.builder instead
> > [WARNING]     val sqlContext: SQLContext = new SQLContext(sc)
> > [WARNING]                                  ^
> > [WARNING] MQTTStreamSourceSuite.scala:138: constructor SQLContext in
> class
> > SQLContext is deprecated: Use SparkSession.builder instead
> > [WARNING]     val sqlContext: SQLContext = new SQLContext(sc)
> > [WARNING]                                  ^
> >
> >
> > - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
> >
> > Best regards,
> > Christian
> >
> >
> >
> >
> > On Thu, Jun 8, 2017 at 8:24 AM, Luciano Resende wrote:
> >
> > > Dear community member,
> > >
> > > Please vote to approve the release of Apache Bahir 2.1.1 (RC2) based on
> > > Apache Spark 2.1.1.
> > >
> > > New Extensions:
> > > * CounchDB/Cloudant
> > > * Google Pub/Sub
> > >
> > > Tag: v2.1.1-rc2 (26bdee0bacb9c922abda09314036e9df0e97d88f)
> > >
> > > https://github.com/apache/bahir/tree/v2.1.1-rc2
> > >
> > > Release files:
> > >
> > > https://repository.apache.org/content/repositories/orgapachebahir-1019
> > >
> > > Source distribution:
> > >
> > > https://dist.apache.org/repos/dist/dev/bahir/bahir-spark/2.1.1-rc2/
> > > <https://dist.apache.org/repos/dist/dev/bahir/2.1.0-rc1/>
> > >
> > >
> > > The vote is open for at least 72 hours and passes if a majority of at
> > least
> > > 3 +1 PMC votes are cast.
> > >
> > >   [ ] +1 Release this package as Apache Bahir 2.1.1
> > >   [ ] -1 Do not release this package because ...
> > >
> > >
> > > Thanks for your vote!
> > >
> > > --
> > > Luciano Resende
> > > http://twitter.com/lresende1975
> > > http://lresende.blogspot.com/
> > >
> >
>
>
>
> --
> Luciano Resende
> http://twitter.com/lresende1975
> http://lresende.blogspot.com/
>

Re: [VOTE] Apache Bahir 2.1.1 (RC2)

Posted by Jean-Baptiste Onofré <jb...@nanthrax.net>.
+1 (binding)

Regards
JB

On Jul 11, 2017, 10:30, at 10:30, Robert Metzger <rm...@apache.org> wrote:
>+1 (binding)
>
>- the staging repo looks good
>- src archive looks good (no binaries, no snapshot dependencies)
>- mvn clean install -DskipTests (including rat license check) passed on
>src
>contents
>
>
>
>
>On Fri, Jun 30, 2017 at 4:03 PM, Luciano Resende <lu...@gmail.com>
>wrote:
>
>> We are still short one vote, any PMC volunteers to review the release
>?
>>
>> On Sat, Jun 17, 2017 at 5:29 PM, Christian Kadner
><ck...@apache.org>
>> wrote:
>>
>> > *+1* (binding)
>> >
>> > The distribution files look good and install build runs
>successfully.
>> >
>> > *Thanks Luciano!*
>> >
>> >
>> > - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
>> >
>> > A few suggestions for future releases:
>> >
>> >
>> > *1)* We may want to enhance the release-build.sh script to update
>the
>> >     *library dependencies* in the module *README.md* files the
>> anticipated
>> >     release version (not "2.2.0-SNAPSHOT" or "2.1.0-SNAPSHOT").
>> >
>> >     Currently none of the README.md files show versions that work,
>i.e.
>> ...
>> >
>> >     ./sql-streaming-akka/README.md:
>> >
>> >     libraryDependencies += "org.apache.bahir" %%
>> "spark-sql-streaming-akka"
>> > % "2.2.0-SNAPSHOT"
>> >
>> >     $ bin/spark-shell --packages
>> > org.apache.bahir:spark-sql-streaming-akka_2.11:2.2.0-SNAPSHOT
>> >
>> >     ./sql-streaming-mqtt/README.md:
>> >
>> >     libraryDependencies += "org.apache.bahir" %%
>> "spark-sql-streaming-mqtt"
>> > % "2.1.0-SNAPSHOT"
>> >
>> >     $ bin/spark-shell --packages
>> > org.apache.bahir:spark-sql-streaming-mqtt_2.11:2.1.0-SNAPSHOT
>> >
>> >     ...
>> >
>> >
>> > *2) *There are *new* *deprecation warnings* we may want to address:
>> >
>> >
>> > [WARNING] CloudantStreaming.scala:67: method registerTempTable in
>class
>> > Dataset is deprecated: Use createOrReplaceTempView(viewName)
>instead.
>> > [WARNING]           changesDataFrame.registerTempTable("
>> > airportcodemapping")
>> > [WARNING]                            ^
>> > [WARNING] PubsubStreamSuite.scala:106: postfix operator
>milliseconds
>> should
>> > be enabled
>> > [WARNING]     eventually(timeout(10000 milliseconds), interval(100
>> > milliseconds)) {
>> > [WARNING]                              ^
>> > [WARNING] PubsubStreamSuite.scala:106: postfix operator
>milliseconds
>> should
>> > be enabled
>> > [WARNING]     eventually(timeout(10000 milliseconds), interval(100
>> > milliseconds)) {
>> > [WARNING]                                                         
>^
>> > [WARNING] PubsubStreamSuite.scala:130: postfix operator
>milliseconds
>> should
>> > be enabled
>> > [WARNING]     eventually(timeout(10000 milliseconds), interval(100
>> > milliseconds)) {
>> > [WARNING]                              ^
>> > [WARNING] PubsubStreamSuite.scala:130: postfix operator
>milliseconds
>> should
>> > be enabled
>> > [WARNING]     eventually(timeout(10000 milliseconds), interval(100
>> > milliseconds)) {
>> > [WARNING]                                                         
>^
>> >
>> >     ... and some *older* ones ...
>> >
>> > [WARNING] MQTTStreamSourceSuite.scala:57: constructor SQLContext in
>> class
>> > SQLContext is deprecated: Use SparkSession.builder instead
>> > [WARNING]     val sqlContext: SQLContext = new SQLContext(sc)
>> > [WARNING]                                  ^
>> > [WARNING] MQTTStreamSourceSuite.scala:128: constructor SQLContext
>in
>> class
>> > SQLContext is deprecated: Use SparkSession.builder instead
>> > [WARNING]     val sqlContext: SQLContext = new SQLContext(sc)
>> > [WARNING]                                  ^
>> > [WARNING] MQTTStreamSourceSuite.scala:138: constructor SQLContext
>in
>> class
>> > SQLContext is deprecated: Use SparkSession.builder instead
>> > [WARNING]     val sqlContext: SQLContext = new SQLContext(sc)
>> > [WARNING]                                  ^
>> >
>> >
>> > - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - -
>> >
>> > Best regards,
>> > Christian
>> >
>> >
>> >
>> >
>> > On Thu, Jun 8, 2017 at 8:24 AM, Luciano Resende wrote:
>> >
>> > > Dear community member,
>> > >
>> > > Please vote to approve the release of Apache Bahir 2.1.1 (RC2)
>based on
>> > > Apache Spark 2.1.1.
>> > >
>> > > New Extensions:
>> > > * CounchDB/Cloudant
>> > > * Google Pub/Sub
>> > >
>> > > Tag: v2.1.1-rc2 (26bdee0bacb9c922abda09314036e9df0e97d88f)
>> > >
>> > > https://github.com/apache/bahir/tree/v2.1.1-rc2
>> > >
>> > > Release files:
>> > >
>> > >
>https://repository.apache.org/content/repositories/orgapachebahir-1019
>> > >
>> > > Source distribution:
>> > >
>> > >
>https://dist.apache.org/repos/dist/dev/bahir/bahir-spark/2.1.1-rc2/
>> > > <https://dist.apache.org/repos/dist/dev/bahir/2.1.0-rc1/>
>> > >
>> > >
>> > > The vote is open for at least 72 hours and passes if a majority
>of at
>> > least
>> > > 3 +1 PMC votes are cast.
>> > >
>> > >   [ ] +1 Release this package as Apache Bahir 2.1.1
>> > >   [ ] -1 Do not release this package because ...
>> > >
>> > >
>> > > Thanks for your vote!
>> > >
>> > > --
>> > > Luciano Resende
>> > > http://twitter.com/lresende1975
>> > > http://lresende.blogspot.com/
>> > >
>> >
>>
>>
>>
>> --
>> Luciano Resende
>> http://twitter.com/lresende1975
>> http://lresende.blogspot.com/
>>