Skip to content

Review "db olympics" results #26661

Open

Description

@alamb

Is your feature request related to a problem or challenge?

@toschmidt of TU-Munich (an influential database research group) posted the following:

Here is the site: https://dbolympics.com/

He invited us to review the configuration of DataFusion to make sure it got a fair evaluation. We currently don't look great in the results:
Image

Describe the solution you'd like

I would like to investigate DataFusions results and make sure we understand (and document) what it would take to improve the system with this benchmark

Given the fact that we are in good (commercial) company with ClickHouse, and the pedigree of the author (the same research lab that published the various query optimizer papers, on which the systems at the top left are based) I am not overly concerned about DataFusion's actual competitiveness in real world queries. However, TUM is influential and I suspect this benchmark will show up in the research literature shortly

I suspect the benchmark skews heavily towards join ordering and systems which use the TUM -inspired algorithms will do well on, but that would be good to confirm.

Describe alternatives you've considered

I think the first thing to do is reproduce the results

Then ideally we would identity the categories were we need to do better. The benchmark site also makes some vague mention that it penalizes correctness but I haven't figured out how that is incorporated yet

Image

Additional context

Related to a similar post recently the polars folks put out that similarly show good join results (TPC-H/TPC-DS):

Willingness to contribute

None

Activity

  1. added theissue type on Oct 23, 2026
  2. alamb commented on Oct 23, 2026

    @alamb
    ContributorAuthor

    One major difference is that DataFusion reads the raw parquet files while all the other systems have to load them first:

    Image

    So you could also spin these results that DataFusion loads 100x faster than other systems. Pointed out here dbolympics/dbolympics#2 (comment)

  3. mbutrovich commented on Oct 23, 2026

    @mbutrovich
    Contributor

    Thanks for sharing this @alamb! I'll dig into the methodology and see how hard it is to reproduce the results locally and/or in the cloud https://dbolympics.com/methodology.html

  4. alamb commented on Oct 23, 2026

    @alamb
    ContributorAuthor

    At the very least I am going to try and reproduce the wrong results they report as part of larger "correctness epic" I working on organizing

  5. alamb commented on Oct 23, 2026

    @alamb
    ContributorAuthor
  6. mbutrovich commented on Oct 23, 2026

    @mbutrovich
    Contributor

    Filed #26671. I will also see if a query rewrite on dbolympics is suitable in the meantime.

  7. alamb commented on Oct 23, 2026

    @alamb
    ContributorAuthor

    Thanks @mbutrovich I have another one related to EXTRACT I am working on

    I also filed a fix for to_char

    Also related to correctness, I filed an epic to track the work

  8. mbutrovich commented on Oct 23, 2026

    @mbutrovich
    Contributor
  9. alamb commented on Oct 23, 2026

    @alamb
    ContributorAuthor

    I also filed a issue about inconsistent EPOCH truncation (which I think is related to some of the db olympics failures)

  10. mbutrovich commented on Oct 23, 2026

    @mbutrovich
    Contributor

    I also filed a issue about inconsistent EPOCH truncation (which I think is related to some of the db olympics failures)

    Nice, thank you! This is the cause of the remaining SF1 wrong results in the benchmark: q124, q127, q128, q197 and q405 all average or take a percentile of CAST(EXTRACT(EPOCH FROM (ts1 - ts2)) AS NUMERIC(18,6)). With the expression replaced by (CAST(ts1 AS BIGINT) - CAST(ts2 AS BIGINT)) / 1000000.0, all five match the reference results. Together with the year_month dialect change and #26671, DataFusion 55.0.0 at SF1 has no wrong results left (331 equal, 0 different, 78 queries that return errors).

  11. mbutrovich commented on Oct 23, 2026

    @mbutrovich
    Contributor

    #26676 is ready for review and delivers speedup on top of the #26673 correctness issue.

  12. comphead commented on Oct 23, 2026

    @comphead
    Contributor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions