Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upGitHub is where the world builds software
Millions of developers and companies build, ship, and maintain their software on GitHub — the largest and most advanced development platform in the world.
get rid of master/slave terminology #5248
Comments
|
maybe:
don't like:
|
|
canonical/alias maybe? |
|
@strugee i know that terminology rather from naming things. but in our case, hardlink names are all equally "good" in the filesystem (only for symlinks, there is a asymmetry). So this is rather about how borg stores these items in the archive. |
|
Yeah, I see what you're saying
The more I think about it the more I like this pairing. It does a pretty good job communicating the idea right in the name, without needing a ton of further explanation. |
|
Not sure why there is a thumbs down on the top post, but maybe it needs some clarification: Of course renaming some words in an open source project does not change history nor does it improve the situation of slaves that still exist in today's world. but keeping such negatively loaded terminology without a good reason or opposing to change them would be even worse, so let's just change them. of course they were never meant in any negative way. |
|
no need to inject divisive politics into open source software |
|
primary/reference, stored/referencing [hardlink], first/following (<-- this is very accurate, but the terms are possibly too generic to be immediately registered as "jargon"). Contentful/contentless is very accurate and descriptive as well, but imho as far as contrasting pairs go both are pretty similar if you squint your eyes.
As far as I can tell this is your first comment on a borg repo, perhaps contribute something else before trying to tell people what to do. |
|
master chunks index -> primary/main chunks index (when referring to .../cache/.../chunks) and reference chunks index (in the case of borg.hashindex.ChunkIndex.stats_against -- the docstring basically calls it this already). Renaming base branches might be a little bit annoying, but I'd suggest |
master/slave was used a lot in the code that deals with hardlinks:
when we archive multiple hardlinks pointing to same inode (file contents), this happens:
sourceattribute points back to the name of the first encountered hardlink. these items do not have a chunkid_list and thus no own content.so we need new terminology for this now, please suggest.