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 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 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 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 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 user ApacheBahir commented on the issue:
https://github.com/apache/bahir/pull/55
Build failed, see build log for details
---
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 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 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 user ApacheBahir commented on the issue:
https://github.com/apache/bahir/pull/55
Build failed, see build log for details
---
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 user ApacheBahir commented on the issue:
https://github.com/apache/bahir/pull/55
Build failed, see build log for details
---
Github user lresende commented on the issue:
https://github.com/apache/bahir/pull/55
ok to test
---
Github user ApacheBahir commented on the issue:
https://github.com/apache/bahir/pull/55
Can one of the admins verify this patch?
---
14 matches
Mail list logo