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

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/08/05 12:37:28 UTC

[GitHub] [pulsar] BewareMyPower edited a comment on pull request #11557: [C++] Fix bugs that were not exposed by broken C++ CI before

BewareMyPower edited a comment on pull request #11557:
URL: https://github.com/apache/pulsar/pull/11557#issuecomment-893422981


   From the output of https://github.com/apache/pulsar/runs/3251560027, we can see all C++ tests passed, and there're some failures or errors of Python tests:
   
   ```
   FAILED (failures=3, errors=4)
   ```
   
   The 3 failures are caused by Python2 incompatibilities, the 4 errors are somehow strange (timeout error when creating producer or consumer) and need further study. @gaoran10 is working on these tests now. Could we merge this PR first and open a new PR to fix Python tests? @merlimat @jiazhai @sijie @codelipenghui 
   
   BTW, this PR doesn't fix the test script, see my comment on https://github.com/apache/pulsar/pull/10309#pullrequestreview-723282954. Once all tests are fixed, we can fix the test script as well.


-- 
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.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

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