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


   > @piotr-szuberski It seems a bit odd but I don't think it's a blocker for 
this PR. In any case, we have a Kafka version in the gradle task name. So if it 
fails then the corresponding task will fail too, right? And we can see that in 
Jenkins job logs, like 
[here](https://ci-beam.apache.org/job/beam_PostCommit_Java_PR/511/consoleText).
   
   Yes, exactly like 
[here](https://ci-beam.apache.org/job/beam_PostCommit_Java_PR/506/console):
   ```
   00:18:25 Execution failed for task 
':sdks:java:io:kafka:kafkaVersion090BatchIT'.
   ```
   > 
   > So, in general it LGTM. Do you have something to add to this PR? 
Otherwise, please undraft it to make it possible to merge.
   
   Great! I've made some cleanup and removed the xml renaming part. When it's 
green then it's ready for merge
   


----------------------------------------------------------------
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:
[email protected]


Reply via email to