this post was submitted on 13 Jun 2024
65 points (100.0% liked)

TechTakes

1427 readers
98 users here now

Big brain tech dude got yet another clueless take over at HackerNews etc? Here's the place to vent. Orange site, VC foolishness, all welcome.

This is not debate club. Unless it’s amusing debate.

For actually-good tech, you want our NotAwfulTech community

founded 1 year ago
MODERATORS
top 50 comments
sorted by: hot top controversial new old
[–] V0ldek@awful.systems 33 points 5 months ago (50 children)

You also definitely shouldn't be using String non-monotonic UUIDs for primary keys in a database, like, literally ever, but what the fuck do I know, I just do databases for a living, I'm not the all-knowing GPT code wizard.

[–] mawhrin@awful.systems 15 points 5 months ago

i'm not doing databases for living but the idea of stringifying a perfectly cromulent unique number in order to store it in the database comes as slightly weird to me.

[–] gnomicutterance@awful.systems 12 points 5 months ago (2 children)

If you’re using a new-to-you ORM, and you don’t ever check the docs to see the basic primary key syntax… it’s SQLAchemy, it’s well documented and there’s tons of prior art.

Also I don’t understand their business case but if a user has a primary key, a unique user ID, and a unique customer ID, then all three of those uniquely identify the customer. (Weird, but there are some plausible explanations.) But then why would you need both the user ID and the customer ID in the subscription table is this some stripe thing I don’t understand or are they just bad at this?

[–] froztbyte@awful.systems 13 points 5 months ago (2 children)

20 bucks the datastructure was designed for easiest access from the semantics of whatever du jour js lib they were using for the app

[–] froztbyte@awful.systems 12 points 5 months ago (1 children)

“designed”, rather

Even “derived from” feels too strong a statement. “Was the result of”?

[–] blakestacey@awful.systems 11 points 5 months ago

"Sediment precipitated from"?

[–] sailor_sega_saturn@awful.systems 11 points 5 months ago

20 bucks their database schema was copy pasted from chat-gpt.

[–] gianni@lemmy.ca 10 points 5 months ago (1 children)

Yes, it's some Stripe thing. Stripe requires you to create a customer to be able to vault payment methods and make charges. However it's possible that not all users in their product require this functionality.

[–] gnomicutterance@awful.systems 8 points 5 months ago

ah, thank you!

load more comments (48 replies)
[–] Architeuthis@awful.systems 26 points 5 months ago* (last edited 5 months ago) (2 children)

Honestly, the evident plethora of poor programming practices is the least notable thing about all this; using roided autocomplete to cut corners was never going to be a well calculated decision, it's always the cherry on top of a shit-cake.

[–] ebu@awful.systems 22 points 5 months ago

the upside: we can now watch "disruptive startups" go through the aquire funding -> slapdash development -> catastrophic failure -> postmortem cycle at breakneck speeds

[–] swlabr@awful.systems 12 points 5 months ago

roided autocomplete

My autocomplete:

  • written in an hour on a whiteboard for a job interview
  • uses only the finest words, hand-selected by unix
  • speed and size efficient

Roided autocomplete:

  • years of development, billions of dollars of investment
  • hallucinates constantly
  • projected to use all computing capacity at current rates
[–] gerikson@awful.systems 22 points 5 months ago* (last edited 5 months ago) (2 children)

The original post was removed, hence the archive link.

HN figures the real issue was the lack of testing/monitoring, not specifically the use of ChatGPT. But the kind of person who's ok with letting spicy autocomplete write their customer acquisition code is probably not the kind of person knowing how to test and monitor.

https://news.ycombinator.com/item?id=40627558

[–] fubarx@lemmy.ml 16 points 5 months ago

I actually tried letting ChatGPT-4o write some tests the other day.

Easily 50% of the tests were wrong. They ignored DB uniqueness constrains or even datatypes. In a few cases, they just hallucinated field names that didn't exist.

I ended up spending just as much time cleaning up the cruft as writing them. I could easily see someone just starting out letting the code go through.

[–] sailor_sega_saturn@awful.systems 14 points 5 months ago

Commit messages are overrated.

Now that's the kind of bad hot take I read awful.systems for! Let's all call ourselves "engineers" but write no documents but emoji laden jokes, and produce no work except for the copy-pasted excreta from a chatbot!

[–] froztbyte@awful.systems 16 points 5 months ago* (last edited 5 months ago) (2 children)

friend of mine happened to dm me this the other day, and here are my reactions from chat, verbatim (albeit timestamps removed)

fucking sfba method, man
so many things in that post, man, god
linear extrapolation for “lost sales”, doing the double presumption of both constant uptake and that everyone would definitely have been a customer
“whatever you’re thinking, double it” bayfucker advice
and just …. everything
fucking hell

and the code. fuck me the fucking code. it's always "nice" to see just how very very clever all the yc fuckers are. but christ does it give me feelings.

but they gotta go fast! if they're barely approaching redshift, are they even agile?!

[–] gnomicutterance@awful.systems 14 points 5 months ago (1 children)

It’s okay to copy/paste your basic model structure for SQLAlchemy classes, but copy and paste from the SQLAlchemy docs. Sweet suffering stack overflow, did nobody even look at the docs ever, or did they only trust ChatGPT? SQLAlchemy‘s simple for basic use cases.

Also here is such a nutshell of everything wrong with YC: jackhole prompt fondlers with no tests, no paying customers, who turn on the most important new feature in prod at the end of the day (jesus wept), and yet with all that clown show,

We had eight ECS tasks on AWS, all running five instances of our backend (overkill, yes we know, but to be fair we had AWS credits).

What the actual fuck.

[–] froztbyte@awful.systems 11 points 5 months ago

Oh yeah I entirely agree about copying - hell, lord knows how much interlang tools suck, I entirely get copycloning some defs best-match

But yeah indeed they super fucked up :D

With a nice cushy bit of “free” to make the problem go away. Whereas recently a fedi admin got their their instance turboscraped and were stressing about a sub-10k bill that would’ve probably killed their entirely service. My contempt for these bayfucker dickweeds knows no bound.

[–] dgerard@awful.systems 12 points 5 months ago

bayfucker advice

perfect

[–] Soyweiser@awful.systems 14 points 5 months ago* (last edited 5 months ago) (1 children)

How did they not find this while testing?

e: Regardless, I don't regret the experience.

eurgh

[–] sailor_sega_saturn@awful.systems 20 points 5 months ago* (last edited 5 months ago)

Even with inadequate tests, inadequate monitoring, and an inadequate integration testing environment; how did it not go like this:

"Hmm the web server appears to be working, but maybe the database is making an oopsie we did just do that whole stupid db migration without a gradual ramp up"

"Oh hey here's an idea, why don't we make sure error messages from the database are logged maybe it's silently crying out in pain right now :D"

load more comments
view more: next ›