+1 (binding)
Regards
JB
On Jul 11, 2017, 10:30, at 10:30, Robert Metzger <rmetz...@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 <luckbr1...@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
><ckad...@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/
>>
