search

apache / incubator-heron

Apache Heron (Incubating) is a realtime, distributed, fault-tolerant stream processing engine from Twitter
https://heron.apache.org/
Apache License 2.0
3.65k stars 599 forks source link

Remove stray space characters between \ and \n #3843

Closed jmtd closed 2 years ago

jmtd commented 2 years ago

The example bash shell commands in this document are wrapped onto multiple lines with backslashes for legibility. However, there are space characters inserted after the backslash escapes, breaking the commands: the spaces are being escaped, not the newlines. An attempt to copy and paste the commands directly from the documentation will fail.

Remove the spaces to fix the commands.

jmtd commented 2 years ago

I haven't yet figured out if/how the documentation at https://hub.docker.com/r/apache/heron is populated, but it suffers the same bug.

joshfischer1108 commented 2 years ago

Good catch and thank you for the fix. I can work to get the docker stuff updated today.

joshfischer1108 commented 2 years ago

@jmtd , please check to verify the issue has been fixed. https://hub.docker.com/repository/docker/apache/heron

jmtd commented 2 years ago

@joshfischer1108 thank you, I can confirm this issue is fixed on the docker hub too

(a separate issue is the example fails - but I'll report that separately once I've played around with it a bit more)

Thanks!

joshfischer1108 commented 2 years ago

Ha! Open source work never ends 😄 . Thank you for looking into it.