Skip to content

Conversation

devnexen
Copy link
Contributor

No description provided.

Signed-off-by: David Carlier <devnexern@gmail.com>
@devnexen devnexen force-pushed the scxtop_pointing_fingers branch from 05e62d3 to 146a9ad Compare July 26, 2025 16:35
@hodgesds hodgesds requested a review from yaakov-stein July 27, 2025 14:31
Copy link
Collaborator

@yaakov-stein yaakov-stein left a comment

Choose a reason for hiding this comment

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

Is this really helpful/necessary? We have limited space at the moment and I think we'd prefer to keep it for scheduler specific information. I can hear an argument for adding in the owning user as a string (root, etc.) but I don't really see the benefit here of adding a u32. Let me know what you think.

Also, for future PR's, it would be great if you could include a screenshot of what the changes look like - thanks!

@devnexen
Copy link
Contributor Author

I can hear an argument for adding in the owning user as a string (root, etc.) but I don't really see the benefit here of adding a u32. Let me know what you think.

I feared the other way around, if user as string, it could possibly take way too much space.

@yaakov-stein
Copy link
Collaborator

I feared the other way around, if user as string, it could possibly take way too much space.

Ya, that's a good point. Maybe let's revisit this in a week or two once a few more of the core features are out for the process view.

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.

2 participants