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

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2020/04/21 05:17:11 UTC

[GitHub] [flink] leonardBang commented on issue #11820: [FLINK-17264][scripts] Fix broken taskmanager.sh by using -eq to compare num_lines

leonardBang commented on issue #11820:
URL: https://github.com/apache/flink/pull/11820#issuecomment-616957287


   > Sorry for introducing this bug in the code. I discover that there are also other scripts (mostly e2e test scripts, for example `test_cli.sh` and `shade.sh`) uses `wc -l` and `==` to compare numbers. Shall we fix them too?
   
   The root cause is that tabs alignment https://apple.stackexchange.com/questions/370366/why-is-wc-c-printing-spaces-before-the-number 
   If these e2e tests fails in macOS(mostly dev environment), I think we should open a jira to track and repair too.


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