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

Reply via email to