+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/
>
