From the amazon I know, people only care about a. not getting fired and b. promotions. For devs, the matrix looks like this:

1. Shipping: deliver tickets or be pipped.

2. Having Less comments on their PRs: for some drastically dumb reason, having a PR thoroughly reviewed is a sign of bad quality. L7 and above use this metric to Pip folks.

3. Docs: write docs, get them reviewed to show you're high level.

Without AI, an employee is worse off in all of the above compared to folks who will cheat to get ahead.

I can't see how "requesting" folks for forego their own self-preservation will work. especially when you've spent years pitting people against each other.

▲

malfist 10 hours ago | parent | next [-]

Not only is having too many comments on your PRs bad for you, but so is not leaving comments on other people's PRs. Both are metrics used

▲

dude250711 7 hours ago | parent [-]

I'd leave lots of comments out of spite whenever I would feel my PRs had been treated unfairly. If I am going down, you all are coming with me.

	▲	grogenaut 4 hours ago \| parent \| next [-]
		I specifically look at the quality / substance of the comments when I'm reviewing someone for promo/transfer/fire.
	▲	malfist 6 hours ago \| parent \| prev [-]
		Welcome to Amazon, you'll fit right in.

▲

embedding-shape 7 hours ago | parent | prev | next [-]

> 2. Having Less comments on their PRs: for some drastically dumb reason, having a PR thoroughly reviewed

I'm very far away from liking Amazon's engineering culture and general work culture, but having PRs with countless of discussions and feedback on it does signal that you've done a lot of work without collaborating with others before doing the work. Generally in teams that work well together and build great software, the PRs tend to have very little on them, as most of the issues were resolved while designing together with others.

	▲	joeframbach 5 hours ago \| parent \| next [-]
		I've been involved in so many CRs where I've given feedback over 10 revs, then the submitter cancels the CR and files a new one, for the metrics.
	▲	tom_ 3 hours ago \| parent \| prev \| next [-]
		If the review tooling is any good, getting the code somewhere it can see it is a convenient way for people to give and receive feedback. As the saying goes, the system is what it does! (And/but yes/no, I have never worked at NAGFAM...)
	▲	ex-aws-dude 6 hours ago \| parent \| prev [-]
		Eh I feel like there are some features where you just have to get in the weeds to even design it and the code review itself is part of the process of designing/figuring out the edge cases.

▲

dboreham 7 hours ago | parent | prev [-]

4. Don't work in the corporate equivalent of The Hunger Games.

	▲	999900000999 5 hours ago \| parent [-]
		At least in the past the idea is you do the dance , vest and leave. I missed my FAANG chance during the good years. No retirement for me!