Facebook engineering generally is super competent. Leet code or not, such slip ups are bound to happen from time to time. True test of competence would be these mistakes don't repeat and that they learn from it, which I'm pretty sure they would.
Exactly. OP makes it harder on everyone expecting engineers to be flawless just because they've gone through a very selective hiring process. Programming is hard.
Lets skip the hyperbole, appeals to authority and platitudes. This crash is from handwritten API response wrapping code that lacks checks. This is string-based programming. This is utter shit.
Ok, I'm sure you've written perfect code your entire career and have never made a single mistake, however simple, and however early in your career. Right?
If you want to criticize something, criticize the testing that let this change through, not the developer who made it. We were all young and inexperienced once, we've all had bad days, and we've all written crap code. This is only unique because it affected a lot of people.
Without staking my flag on either side of this argument, it's a bit silly to say "the only reason this stands out is because it affected so many people" as if that's a reasonable counter-argument.
The large amount of people affected reinforces your parent comment's argument. Something that has the potential to affect an immense amount of people should be treated with a relative amount of care. Writing crap code and pushing carelessly would be a far more significant failure of judgement if that code were in the Facebook SDK than if it were in Johnny's Hello World App.
It's also worth noting the difference between writing code with a minor change in behavior that when pushed to production causes an unexpected domino-effect of obfuscated cascading failures across multiple different services that eventually impacts an immense amount of people, vs. directly writing code that crashes on the client side and impacts an immense amount of people. This is the latter.
Why was the testing not sufficient? Was it even tested at all? Maybe a developer just "pushed it straight to master"? Why/how can they do that (hypothetically)
It has everything to do with it. Software engineers should take care to consider the risks and potential downsides to the shady products they build with that talent that aren't just downtime or tech debt but the larger impact to society itself. Silicon Valley prioritizes technical talent while completely disregarding strength and quality of character.
> “ True test of competence would be these mistakes don't repeat and that they learn from it, which I'm pretty sure they would.”
Which begs the question why test engineers with leetcode trivia bullshit. You are yourself emphasizing why their interview process (and they are just one of many offenders) is so bananas inappropriate.
One single bug is why their process is bananas inappropriate? I really hope all of your code is perfect every single time to be acting so high and mighty.
Those criticisms aren't about mere bugs, but a bloated over-engineered iOS apps with far too many classes (18k back in 2015) [0]. The Android app was also previously notorious for patching Dalvik at runtime to deal with the huge number of Java methods [1].
Granted, perhaps things have gotten better in the last five years.
It's certainly possible for an enormous engineering company to have poor code quality or poor engineering practices, despite having a high interviewing standard. And a criticism of Facebook need not be a criticism of all companies that interview by Leetcode, for instance no one here is criticizing Google, Amazon, or Apple, and all have better reputations for code quality.
You’re describing the attitude Facebook takes towards candidates, not the attitude I take towards anything.
Candidates should be saying to Facebook, “ I really hope all of your code is perfect every single time to be acting so high and mighty” and absolutely making a huge deal out of a case like this when it isn’t.
Your comments have taken a noticetable turn to the trollish lately. If that keeps up, we're going to ban you again. Can you please review https://news.ycombinator.com/newsguidelines.html and course-correct?
The vast majority of indian immigrants in the us spend some years in higher education in the us where most of those British English/Indian English phrases are trained away. And, only the cream of the cream of the crop is really able to immigrate, especially with current immigration policies. So, when you're referring to incompetent coders that say "do the needful," the only way that makes sense is if you are referring to second rate outsourcing companies overseas. It wouldn't make very much sense to imply that the problems in Facebook's SDK arise because of a group of highly skilled immigrants and second generation american born desis with graduate training from top universities, would it?
