[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...

2017-12-07 Thread ApacheBahir
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...

2017-12-07 Thread ApacheBahir
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...

2017-12-07 Thread ckadner
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]

[GitHub] bahir pull request #56: Dummy test to force build failure (DONT MERGE)

2017-12-07 Thread ckadner
GitHub user ckadner opened a pull request: https://github.com/apache/bahir/pull/56 Dummy test to force build failure (DONT MERGE) This is a test of our Jenkins PR builder setup. **DON'T merge this PR!** You can merge this pull request into a Git repository by running:

[GitHub] bahir issue #56: [BAHIR-150] Test Jenkins PR build config changes (DONT MERG...

2017-12-07 Thread ckadner
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/56 After adding the `--fail-at-end` flag to the PR builder's maven build, now all modules get built and tested even after the forced test failure in module `sql-cloudant`: ``` [INFO]

[GitHub] bahir pull request #56: [BAHIR-150] Test Jenkins PR build config changes (DO...

2017-12-07 Thread ckadner
Github user ckadner closed the pull request at: https://github.com/apache/bahir/pull/56 ---

[GitHub] bahir issue #56: [BAHIR-150] Test Jenkins PR build config changes (DONT MERG...

2017-12-07 Thread ckadner
Github user ckadner commented on the issue: https://github.com/apache/bahir/pull/56 retest this Jenkins ---

[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...

2017-12-07 Thread ckadner
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 #56: [BAHIR-150] Test Jenkins PR build config changes (DONT MERG...

2017-12-07 Thread ApacheBahir
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/56 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/109/ ---

[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...

2017-12-07 Thread ckadner
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 pull request #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-07 Thread emlaver
GitHub user emlaver opened a pull request: https://github.com/apache/bahir/pull/57 [BAHIR-128][WIP] fix failing sql-cloudant test _What_ Fix test that's failing sporadically in sql-cloudant's `CloudantChangesDFSuite`. _How_ - Call stop in receiver when _changes

[GitHub] bahir pull request #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-07 Thread emlaver
Github user emlaver commented on a diff in the pull request: https://github.com/apache/bahir/pull/57#discussion_r155699572 --- Diff: pom.xml --- @@ -458,7 +458,7 @@ .gitignore .repository/ - .idea/ +

[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...

2017-12-07 Thread ckadner
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

[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-07 Thread ApacheBahir
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Build successful ---

[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...

2017-12-07 Thread ckadner
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

[GitHub] bahir pull request #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-07 Thread ckadner
Github user ckadner commented on a diff in the pull request: https://github.com/apache/bahir/pull/57#discussion_r155697683 --- Diff: pom.xml --- @@ -458,7 +458,7 @@ .gitignore .repository/ - .idea/ +

[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-07 Thread ApacheBahir
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Build successful ---

[GitHub] bahir issue #57: [BAHIR-128][WIP] fix failing sql-cloudant test

2017-12-07 Thread ApacheBahir
Github user ApacheBahir commented on the issue: https://github.com/apache/bahir/pull/57 Refer to this link for build results (access rights to CI server needed): http://169.45.79.58:8080/job/bahir_spark_pr_builder/111/ ---

[GitHub] bahir issue #55: [BAHIR-104] Multi-topic MQTT DStream in Python is now a Pai...

2017-12-07 Thread emlaver
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...

2017-12-07 Thread ckadner
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. ---