[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/55 Any further comments? @anntinutj - you commented on [BAHIR-104](https://issues.apache.org/jira/browse/BAHIR-104) @fbeneventi - I saw you worked on this before (https://github.com/fbeneventi/bahir/commit/3755ecc) ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/55 **LGTM** I ran the (new) Python tests manually and they completed successfully. **Before:** ``` [bahir] (master *=)$ streaming-mqtt/python-tests/run-python-tests.sh -- Ran 1 test in 22.871s OK ``` **After:** ``` [bahir] (pr-55_BAHIR-104_python_pairRDD *=)$ streaming-mqtt/python-tests/run-python-tests.sh -- Ran 2 tests in 27.593s OK ``` ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/55 @zubairnabi-intech -- we can ignore the build failure for now, but I still need to manually test your changes. ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/55 We still have failing tests in `sql-cloudant` but all other modules were built and tested successfully. ``` [INFO] [INFO] Reactor Summary: [INFO] [INFO] Apache Bahir - Parent POM .. SUCCESS [ 4.356 s] [INFO] Apache Bahir - Spark SQL Cloudant DataSource ... FAILURE [03:13 min] [INFO] Apache Bahir - Spark Streaming Akka SUCCESS [ 27.906 s] [INFO] Apache Bahir - Spark SQL Streaming Akka SUCCESS [03:49 min] [INFO] Apache Bahir - Spark Streaming MQTT SUCCESS [01:53 min] [INFO] Apache Bahir - Spark SQL Streaming MQTT SUCCESS [02:05 min] [INFO] Apache Bahir - Spark Streaming Twitter . SUCCESS [ 25.999 s] [INFO] Apache Bahir - Spark Streaming ZeroMQ .. SUCCESS [ 18.989 s] [INFO] Apache Bahir - Spark Streaming Google PubSub ... SUCCESS [ 53.220 s] [INFO] Apache Bahir - Spark Extensions Distribution ... SUCCESS [ 3.500 s] [INFO] [INFO] BUILD FAILURE [INFO] [INFO] Total time: 13:16 min [INFO] Finished at: 2017-12-07T17:02:57-08:00 [INFO] Final Memory: 183M/6960M [INFO] ``` ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/55 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/110/ ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/55 Build failed, see build log for details ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/55 I changed the Jenkins build configuration to continue the build after failed modules (#56) ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/55 Actually, our Maven build does not kick of the Python tests, so we may have to test this PR "manually" for the time being. I will start on making the build changes independently. ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user emlaver commented on the issue: https://github.com/apache/bahir/pull/55 @ckadner Hi yes I will look into fixing this sporadically failing test. This shouldn't hold up this PR, right? ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/55 Build failed, see build log for details ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user zubairnabi-intech commented on the issue: https://github.com/apache/bahir/pull/55 The build failure is actually unrelated to this change. A Cloudant test in Scala is failing. ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/55 Build failed, see build log for details ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user lresende commented on the issue: https://github.com/apache/bahir/pull/55 ok to test ---
[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/55 Can one of the admins verify this patch? ---