6 ms·
But all of the engineers passed LeetCode, how could this be?
by ExactActuation 6y ago
But all of the engineers passed LeetCode, how could this be?
- himaraya 6y agoQuestions went stale
- ignoramous 6y agoFacebook 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.
- mlthoughts2018 6y ago> “ 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.
- pb7 6y agoOne 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.
- mlthoughts2018 6y agoThis is so shortsighted it’s painful. 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.
- pb7 6y agoYou are making a MASSIVE assumption that most people dislike Leetcode. I personally don't and neither does anyone I know. It's way better than the interview formats of almost every other profession out there that's 2 parts sucking up, 2 parts connections, and 1 part luck. Show me a better process that can handle 100,000+ candidates a year with a stronger signal and less or equal time invested and I'll change my mind. At the very least, hating the current process shows that you're lazy, stubborn, and unwilling to learn what is necessary to get the job done. Hardly desirable traits in a coworker.
- mlthoughts2018 6y ago> “ You are making a MASSIVE assumption that most people dislike Leetcode.” No, this is not an assumption.
- Apocryphon 6y agoThis isn't exactly the first time Facebook has been called out for poor engineering practices: https://www.darkcoding.net/software/facebooks-code-quality-problem/ https://www.darkcoding.net/software/facebooks-code-quality-p... https://blog.timac.org/2017/0410-analysis-of-the-facebook-app-for-ios-v-87-0/ https://blog.timac.org/2017/0410-analysis-of-the-facebook-ap...
- pb7 6y agoSo what? They're an enormous engineering company with many millions of lines of code. Bugs are expected.
- Apocryphon 6y agoThose 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. [0] https://www.reddit.com/r/iOSProgramming/comments/6upeu6/how_on_earth_is_the_facebook_app_size_so_large/dluf4rc/ https://www.reddit.com/r/iOSProgramming/comments/6upeu6/how_... [1] https://www.facebook.com/notes/facebook-engineering/under-the-hood-dalvik-patch-for-facebook-for-android/10151345597798920 https://www.facebook.com/notes/facebook-engineering/under-th...
- saagarjha 6y agoI think I’ve done things like what Facebook did, when I was writing one-off security exploit scripts. Someone needs to stop that team.
- 5cott0 6y agoSo you're saying we can be confident they won't break democracy again?
- airstrike 6y agohttps://www.quitfacebook.org/img/zucc-ads-smile-robot-small.gif https://www.quitfacebook.org/img/zucc-ads-smile-robot-small....
- pb7 6y agoWhat does that have to do with engineering talent?
- 5cott0 6y agoIt 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.
- Lammy 6y ago> True test of competence would be these mistakes don't repeat and that they learn from it It's not that they aren't smart or don't learn but that FB's performance review process rewards optimizing for growth over reliability/sustainability. e: you can downvote me but I lived it so you won't change my mind :)
- SwiftyBug 6y agoExactly. 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.
- stefan_ 6y agoLets 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.
- mynameisvlad 6y agoOk, 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.
- madeofpalk 6y ago> criticize the testing Criticise the process that let it through. 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)
- rdgthree 6y agoWithout 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.
- akmarinov 6y agoThat’s not a slip up on their end, it’s a feature to enable them to collect stuff even if you don’t use their code.
- deleted 6y ago[deleted]
- empath75 6y agoIf only they knew how to balance a binary search tree.
- vmception 6y agothe product manager began telling engineers to "do the needful" gotta localize the English to get the point across
- vikramkr 6y agoFacebook doesn't seem like the kind of company that would be outsourcing their SDK development to some random Indian outsourcing firm stuffed with graduates from second-tier engineering schools that don't know their : from their ;
- vmception 6y agooh I didn't imply outsourcing, they're in the bay area
- vikramkr 6y agoThe 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? Would it?
- vmception 6y agolol, alright, thanks for taking the moment to educate
- dang 6y agoYour 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 https://news.ycombinator.com/newsguidelines.html and course-correct?
- 5cott0 6y agoLeetCode easy