You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@bahir.apache.org by GitBox <gi...@apache.org> on 2021/05/11 21:30:58 UTC

[GitHub] [bahir-flink] raminqaf edited a comment on pull request #114: [BAHIR-274] Add Flink InfluxDBv2.0 Connector

raminqaf edited a comment on pull request #114:
URL: https://github.com/apache/bahir-flink/pull/114#issuecomment-838786735


   @rmetzger Everything should be ready and set for the merge now! :rocket:  only one small thing:
   I noticed that the Travis CI Pipeline is failing (for both environments, scala 2.11 & 2.12) due to three tests in the `flume` connector. Here is a screenshot of the Travis logs:
   ![image](https://user-images.githubusercontent.com/20357405/117848190-561af800-b283-11eb-9bde-e4946202793f.png)
   I found a quick fix for it: I just updated to the newest version of `testcontainers`, and it seams that this fixes the problem. I created a [PR](https://github.com/apache/bahir-flink/pull/123) for this issue. The pipeline passes successfully. I didn't know how to create an issue on Jira but maybe you can help me out on that :) Also, I couldn't add anyone to the reviewers on my PR.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org