[GitHub] [beam] piotr-szuberski commented on pull request #12847: [BEAM-7523] Fix starting Kafka container twice in KafkaCSVTableIT

2020-09-17 Thread GitBox 


piotr-szuberski commented on pull request #12847:
URL: https://github.com/apache/beam/pull/12847#issuecomment-694099897


   > Sorry to hear about the food poisoning, hope you're feeling better now :)
   
   I was up and running the next day, thank you :)



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:
us...@infra.apache.org

[GitHub] [beam] piotr-szuberski commented on pull request #12847: [BEAM-7523] Fix starting Kafka container twice in KafkaCSVTableIT

2020-09-15 Thread GitBox 


piotr-szuberski commented on pull request #12847:
URL: https://github.com/apache/beam/pull/12847#issuecomment-693204054


   Run SQL PostCommit



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:
us...@infra.apache.org

[GitHub] [beam] piotr-szuberski commented on pull request #12847: [BEAM-7523] Fix starting Kafka container twice in KafkaCSVTableIT

2020-09-15 Thread GitBox 


piotr-szuberski commented on pull request #12847:
URL: https://github.com/apache/beam/pull/12847#issuecomment-693203823


   > Looks like moving the imports around broke spotless, I'll push a commit to 
fix so we can get this merged today.
   
   Sorry, I even had this commited locally yesterday but in the meantime I got 
a food poisoning and just forgot to push it.



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:
us...@infra.apache.org

[GitHub] [beam] piotr-szuberski commented on pull request #12847: [BEAM-7523] Fix starting Kafka container twice in KafkaCSVTableIT

2020-09-15 Thread GitBox 


piotr-szuberski commented on pull request #12847:
URL: https://github.com/apache/beam/pull/12847#issuecomment-692544119


   @TheNeuralBit Could you review this? I think it's worth to have it done asap.



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:
us...@infra.apache.org