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

Adding ability to truncate output #5106

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ShawnTuatara
Copy link

If there are too many tcp connections then the top doesn't work well as it scrolls the shell window. This -o option lets you limit the output to however many lines you want.

If there are too many tcp connections then the top doesn't work well as it scrolls the shell window. This `-o` option lets you limit the output to however many lines you want.
@Bojun-Seo
Copy link
Contributor

Can you explain the differences between -o option and position argument count option?

And it will be helpful if you add some examples about
(1) What you cannot do without the -o option
(2) What you can do with the -o option

@ShawnTuatara
Copy link
Author

Count is how many times to loop through and output all the connections. The output lines reduces how many lines to show for each loop. When the output lines isn't specified then if there is more connections than screen / terminal lines the top of the list, and header lines, scrolls off the screen. In most cases users only want to see the top connections along with the header information.

@Bojun-Seo
Copy link
Contributor

Thanks for the explanation.

There is a -T, --top option in syscount, argdist, memleak and ppchcalls tools which is similar feature in this patch.
So I suggest to change the option name from -o to -T as other tool does.

@yonghong-song
Copy link
Collaborator

Agree. option -T is more consistent with other tools.

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.

3 participants