I don't understand why all these problems should be easier handled with an ORM then with raw sql?

Why is it so hard to believe that well tested, typed code is better than manual string concatenation?

Before you tell me about how you just use a Query Builder/DSL and a object mapper for convenience: That's a freaking ORM!

▲

tossandthrow 5 days ago | parent | prev [-]

It is a granluarity tradeoff.

With SQL you need to explicitly test all queries where the shape granularity is down to field level.

When you map data onto an object model (in the dto sense, not oop sense) you have bigger building blocks.

This gives a simpler application that is more reliable.

Obviously you need to pick a performant orm - and it seems a lot of people in these threads have been traumatized.

Personally, I run a complex application where developers freely use a graphql schema and requests are below 50ms p99 - gql in translated into joins by the orm, so we do not have any n+1 issues, etc.

▲

johnmaguire 5 days ago | parent | next [-]

The issue with GraphQL tends to be unoptimized joins instead. Is your GraphQL API available for public consumers? How do you manage them issuing inefficient queries?

I've most often seen this countered through data loaders (batched queries that are merged in code) instead of joins, or query whitelists.

▲

tossandthrow 5 days ago | parent [-]

While this api in particular is not publicly exposed, that would not be a concern.

The key is to hold the same schema on the database as on the graphql and use tooling that can translate a gql query into a single query.

▲

johnmaguire 5 days ago | parent [-]

The issue I've seen with GraphQL isn't necessarily the count of queries run, but rather the performance or said queries (i.e. most SQL queries are not performant without proper indexes for the specific use case, but GraphQL allows lots of flexibility in what queries users can run.)

	▲	tossandthrow 3 days ago \| parent [-]
		Yes - one needs to ensure that the data is well indexed - that is reasonable. But indices does not need to yield a single result. It is OK that indices reduce the result set to tens or couple of hundreds of result. That is well within the performance requirements (... of our app)

▲

Yokohiii 5 days ago | parent | prev | next [-]

In my ears that's just neglect? You assume your ORM does the basic data mapping right and don't verify it?

▲

marcosdumay 5 days ago | parent | next [-]

> You assume your ORM does the basic data mapping right

You know, it should. There's no good reason for an ORM to ever fail at runtime due to mapping problems instead of compile time or start time. (Except, of course if you change it during the software's execution.)

	▲	Yokohiii 5 days ago \| parent [-]
		Why should a raw query fail?

▲

tossandthrow 5 days ago | parent | prev [-]

No? The difference is to verify it ones for the orm VS ones for every single place your query.

▲

Yokohiii 5 days ago | parent | next [-]

I have to respond here as I seemingly the depth limit is reached.

As you've mentioned graphql you probably comparing ORM in that sense to an traditional custom API with backed by raw sql. In a fair comparison both version would do the exactly same, require the same essential tests. Assuming more variations for the raw sql version is just assuming it does more or somehow does it badly in terms of architecture. Which is not a fair comparison.

	▲	tossandthrow 5 days ago \| parent [-]
		The orm represents deferred organization. Ie someone else is testing mapping and query generation for you. An example is prisma. Prisma has a team og engineers that work on optimizing query generation and provide a simple and intuitive api. Not using an orm forces you to take over that organization and test that extra complexity that goes into you code base. It might be merited if you get substantiel performance boosts - but I have not seen any reasonably modern orm where performance is the issue.

▲

Yokohiii 5 days ago | parent | prev [-]

A raw query doesn't has to be repeated in every place it's required. Not sure what your point is.

	▲	tossandthrow 5 days ago \| parent [-]
		You will have a bigger variety of queries hwne you don't use an orm - this puts a higher load on software testing to get the same level of reliability.

▲

5 days ago | parent | prev | next [-]

[deleted]

▲

sgarland 5 days ago | parent | prev [-]

> 50 ms p99

You realize that’s abysmally bad performance for any reasonable OLTP query, right? Sub-msec (as measured by the DB, not including RTT etc.) is very achievable, even at scale. 2-3 msec for complex queries.

	▲	tossandthrow 3 days ago \| parent [-]
		The is the response time for the server, not the database - which is appears that everyone but you understood clearly from the context.