Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

reduce noisy log messages from wget #2656

Merged
merged 1 commit into from
Aug 15, 2018

Conversation

raulchen
Copy link
Contributor

Currently, wget prints tons of progress messages (roughly 1900 lines, see below), which makes the log hard to read.
This is because, by default, wget prints a dot for each 10k received data. With this change, it prints a dot for every 1M data.

         0K .......... .......... .......... .......... ..........  0% 60.2K 26m20s
        50K .......... .......... .......... .......... ..........  0%  171K 17m47s
       100K .......... .......... .......... .......... ..........  0%  257K 13m54s
       150K .......... .......... .......... .......... ..........  0%  330K 11m37s
       200K .......... .......... .......... .......... ..........  0%  438K 10m1s
       250K .......... .......... .......... .......... ..........  0%  503K 8m52s
       300K .......... .......... .......... .......... ..........  0%  483K 8m4s
       350K .......... .......... .......... .......... ..........  0%  815K 7m17s
       400K .......... .......... .......... .......... ..........  0%  554K 6m48s
       450K .......... .......... .......... .......... ..........  0%  454K 6m28s
...

@AmplabJenkins
Copy link

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Ray-PRB/7486/
Test PASSed.

Copy link
Contributor

@pcmoritz pcmoritz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, this is great! Maybe we can do something similar for pip, it currently also prints a lot of progress.

@pcmoritz pcmoritz merged commit 3c75e71 into ray-project:master Aug 15, 2018
@raulchen raulchen deleted the wget_progress branch August 16, 2018 02:52
@raulchen
Copy link
Contributor Author

@pcmoritz created #2667

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants