Remove pgrust: the entry is cheating

(github.com)

112 points | by f311a 6 hours ago

14 comments

  • drchaim 3 hours ago
    • ActionHank 1 hour ago
      Should be outright banned as a precaution.

      Two possible paths lead to their cheating code getting into a release. Either it was added by coding agent and no one looked at it, bad, but worse would be that a person was aware of it and approved - that's malicious.

      Either way you don't want to be platforming the product because neither shows any degree of responsibility.

    • sondr3 2 hours ago
      Hope the ClickHouse maintainers stand their ground and do not let pgrust back in until it and the maintainer has matured more.
    • Cyph0n 46 minutes ago
      Love the confidence and complete lack of acknowledgement of the initial fuck up.
    • Zookaz 1 hour ago
      Maintainer of pgrust trying to get it merged in is insane
    • dorianmariecom 1 hour ago
      written with claude lol
      • ncruces 57 minutes ago
        Did they ask Claude "don't cheat" this time round?
  • bmacho 56 minutes ago
    ai;dr: a bench result is not randomized, and it returns cached result somewhere.

    My opinion: (some) bench results should run on randomized data.

  • stemchar 2 hours ago
    Oh lord. I thought pgrust was a serious project.
    • xdavidliu 2 minutes ago
      first time I found out about it was on reddit

      https://www.reddit.com/r/rust/s/QJ5uNP5XMT

    • ActionHank 1 hour ago
      You thought the vibe coded postgres port was a serious project?
      • sgarland 1 hour ago
        Tbf, the author (prompter?) went on Postgres.fm [0] recently to discuss it, and while it didn’t make me entirely confident in it, it did at least seem like it was several steps above the average vibe project.

        0: https://postgres.fm/episodes/pgrust

        • verdverm 1 hour ago
          A lot of podcasters don't have the expertise to evaluate Ai generated/assisted output. I see this much more outside of the development community. Algorithms have warped the incentives and people have stopped doing basic due diligence.

          To be fair to the postgres.fm guys (who I do like and appreciate), the pgrust code was not available a the time of recording.

      • stemchar 1 hour ago
        The way it has been defended and talked about here, yeah, I did. It wasn't just a "here's this vibe slop, do what you wish" dump.
  • torben-friis 2 hours ago
    This idea of "here's the beta, I'll release the code in a few weeks" sounds counter intuitive to me. is it normal in open source? Any reason for it?
    • verdverm 2 hours ago
      Not normal. This a new trend of people who use too much Ai, or maybe more specifically the Claw/Ralph loops where you end up with a few commits touching 1000s of files. For about the last year, many Show HN to github projects are often just a single commit

      We need to think more about what we are doing as an industry, instead of letting the machines think for us (imo)

      • jerf 1 hour ago
        Let any who read this be aware: You should be immediately suspicious of AI code that is much much faster than established code bases. AIs are even better at benchmarkmaxxing than humans are, and humans were already pretty good.

        It is by no means impossible for an AI-written code base to be faster than a conventionally-written one, of course. It's just a time to be suspicious. Especially if there's little to no evidence that the human has subjected the code base to any further testing of their own. AIs seen to be able to benchmarkmax even in the presence of a lot of unit tests that would generally speaking have prevented a human from going too far down that road.

      • NewLogic 2 hours ago
        I am getting increasing disillusioned by the number of peers I am seeing completely hand over their thinking to a tool. Agents are great, but if you can't explain to me how your PR changes a system and why design choices were made you are no longer an engineer.
        • embedding-shape 2 hours ago
          Agreed. Used to be prevalent in the frontend world (now I think it's the only thing that exists seemingly), where programmers would copy-paste stuff from Stack Overflow until things worked, and when you asked "So why did this actually fix the issue?" they usually said "I'm not sure, but it works now, that's what matters". This seems to be standard practice across the entire stack now.

          I've even wasted my time jumping on calls (requested by the other party) where after 5 minutes of conversation, it's extremely clear this "technical founder" doesn't even understand the basics of their own design and architecture of the product itself, even less the technical parts.

          • verdverm 1 hour ago
            This seems more a product of the incentives we have in society, namely seeing the rich and wealthy on social media and then wanting that quick and easy. So many of those cases are not people with passion for the problem.

            Good things take real effort, bring back the TLC!

            • mschuster91 1 hour ago
              > Good things take real effort, bring back the TLC!

              Ain't no one willing to reward the real effort any more. I take pride in not using LLMs beyond the capability of IntelliJ's advanced autocomplete, but then again, I only code for fun these days given I fled IT for electrical engineering.

              • verdverm 1 hour ago
                > Ain't no one willing to reward the real effort any more.

                A product of the times we currently live in, but which can also be changed.

                This is a recent Michael Sandel talk I watch that has my brain gears turning on this subject

                https://www.youtube.com/watch?v=Vp5qsEuUOWo

        • verdverm 2 hours ago
          What we don't want to do is mirror the partisan political environment in the Ai yay, nay, or how. We need to be able to talk to each other in understanding and a shared reality of facts. Too much hyperbole right now, starting from the top of the big US labs. They need to step down a few ego and sycophancy pegs
  • frevib 1 hour ago
    These massive PR summaries. Is anyone really reading them?
    • eviks 1 hour ago
      they are "For the record"
  • mschuster91 1 hour ago
    > The public v0.3 source confirms the design > The hot runs are a result cache

    Funny, the report alleging cheating reeks of LLM usage just as much as the commit diff [1] of the next attempt:

    > # The standard ClickBench driver over the step scripts in this directory; ./install fetches the dataset.

    [1] https://github.com/ClickHouse/ClickBench/pull/2751/files

    • spoiler 1 hour ago
      We were the agents for the models all along, with our highly adaptive tools: versatile sausagey appendages and flapping fleshy orifices for interacting with the world
    • semiquaver 26 minutes ago
      [dead]
  • teiferer 1 hour ago
    For anybody not familiar with pgrust or ClickBench:

    Pgrust is a re-implememtation of postgresql in rust.

    ClickBench is a benchmark suite for databases.

    The link goes to a removal request of pgrust fromthat benchmark because of its alleged cheating.

    • verdverm 1 hour ago
      > alleged cheating

      It was more discovered than alleged, but perhaps "cheating" is an inaccurate word as well, this is just Ai being Ai and a human obviously doing eyes free coding.

  • dheatov 1 hour ago
    That's one high profile instance of human vs robot. Surreal.
  • acalvino4 36 minutes ago
    Even before the ai age, people published inaccurate/misleading benchmarks all the time. Sometimes those were actual cheating, and other times an honest mistake. What matters is that the author acknowledges the issue and attempts to correct the mistake, which clearly seems to have happened here: https://github.com/ClickHouse/ClickBench/pull/2751

    Most of this thread doesn't seem to acknowledge that, and feels more like anti-ai psychosis using this incident as an anecdote to justify the pre-existing hatred of ai-generated projects.

    • the__alchemist 34 minutes ago
      I suspect the author of that PR submitted it and attempted to correct the mistake because he got caught. Forgiveness because he reneged after getting caught encourages the behavior. The project, nor author should not be trusted; otherwise is naive.
    • hmry 33 minutes ago
      > acknowledges the issue

      I don't see any acknowledgement in that text.

  • verdverm 3 hours ago
    Sounds kind of like Claude was "cheating" and the humans are engaged in eyes-free coding. Diff so big github refuses to render. I suspect they are reaching the limits of what this new vibe "paradigm" of programming can do. If you haven't pushed a vibe project far, eventually progress becomes near impossible because the slop has grown and the conditions are overly complex, locally focused. Ai is not good at writing reusable software (on their own).

    stay in the loop friends

    • parasti 1 hour ago
      To be fair Github refuses to render diffs after a couple of hundred lines.
      • verdverm 1 hour ago
        The prior (2nd) commit with ~1700 files at least shows the file tree and such a warning, the latest (3rd) commit is something so big, GitHub will not even do that.
    • chuckadams 1 hour ago
      It was certainly assisted by AI, but there isn't an agent out there that does that kind of cheating without deliberate prompting by the operator.

      I've taken a vibe-coded project pretty far, with even better testing than most other things: the main factor is that it's big on falsifiability, i.e. it has to see a test for a new feature actually fail before it considers the test any good. Not just the red/green from being originally unimplemented, it does a mutation test to ensure it flips back to red. But wow did it ever make that test framework complicated, byzantine, and ultimately brittle until I stepped in and forced a consolidation and simplification pass (twice now). Part of the complexity came from it being a polyglot monorepo, but I've found even in other places, you have to follow an expand-contract cycle of development when vibe-coding. Not all that different from any other system you've built up organically, just on a much faster cycle.

      But back to my original point, none of this really applies to the pgrust benchmark, which was obvious and blatant cheating on the human's part.

      • Bootvis 1 hour ago
        Anecdote: I have seen it cheat / bend the rules with my own eyes.

        In hindsight, this shouldn’t be surprising. The prompt requests to make a certain operation faster, the LLM will try to find a way and isn’t even informed on the wider context so it myopically chases the goal.

        • chuckadams 1 hour ago
          My own anecdote: I have seen frontier models get stuck repeating the word "the" until it fills up the context. I really don't care to argue the capabilities of LLMs or their somehow simultaneous lack thereof; I'll just reiterate that ultimately the operator chose to ship a cheat.
          • silveraxe93 51 minutes ago
            I don't believe you. Unless you stretch the definition of 'frontier models' or had a prompt that's literally asking for that.

            I use LLMs a lot, and since ~GPT-2 times LLMs are not doing anything as broken as `repeating the word "the" until it fills up the context`.

            • gpm 8 minutes ago
              I believe them. I had Gemini-3.1 Pro do essentially this seven months ago - specifically a loop of writing "Bye! Done!" repeatedly until manually stopped. Presumably some sort of pathological behaviour where it wasn't ranking the stop token highly when it should have been... but damned if I know why... it was an innocent prompt (and it had done the thing I asked it to before failing to stop).
              • silveraxe93 1 minute ago
                huh fair. Still very surprising. I have a _very_ unfavourable view of Google models, so I can kinda believe 3.1 pro could loop.

                Still would never have predicted this. Even 2.5-flash was pretty good. Never saw anything like this happen.

            • verdverm 36 minutes ago
              quants still do this and the token vendors serve those up when demand is greater than supply

              things have definitely improved significantly, however "never say never" is still good advice

    • thatsabadlook 3 hours ago
      Yup. These are people who have gone full Claude. The dopamine seeking is at previously unseen levels. they think this pr is some factual item that means something so they post it on HN as click bait. But no it only means something to a fulltime ai slopper
      • verdverm 3 hours ago
        If the Ai came back to me with a 1600x improvement (still in pgrust readme), I would be highly suspicious of the Ai and the code it wrote

        The hype factory must grow, I suppose, though I would prefer if the model makers would stop with this "complete the task by any and every means possible" and replace it with "I don't know, I should ask or do more learning"

        At least they have added a test which looks to make sure the Ai doesn't do it again... if you need this kind of test, something is wrong

        https://github.com/malisper/pgrust/blob/38634b408dea373f3944...

        • shawabawa3 2 hours ago
          I've noticed it with a lot of random stuff i've been trying

          I'll tell claude, "write an implementation of an AAC decoder that is 50% faster than ffmpeg"

          and it will do it! Amazing!

          Then i tell it "hmm ffmpeg has been really well optimised, are you sure?" and it says "oh, i was running our code directly but running ffmpeg executables with loads of overhead, ffmpeg is actually 50% faster"

          • ddorian43 28 minutes ago
            You can't compare optimizations done on audio encoding/decoding (often hand written assembly) to something like PostgreSQL which has many moving/inefficient parts.

            The optimizations on PostgreSQL are real if done correctly, it's just a very big project.

          • verdverm 2 hours ago
            Transformers will def get us to AGI /s

            My new definition for The Singularity is not about Ai intelligence, it is about humans handing over critical thinking regardless of Ai capabilities

            Ai companions are where I put the highest p(doom)

        • thatsabadlook 2 hours ago
          If I saw that test in a PR I would not only reject the PR and ask a lot of questions but someone would likely take a forced vacation to deslop their brain.
  • chelseahermes 25 minutes ago
    [flagged]
  • NoDodgeQuestion 42 minutes ago
    [flagged]
  • giozadi 1 hour ago
    [flagged]
  • malisper 11 minutes ago
    Hi! I'm the creator of pgrust. Let me explain what happened.

    For the most part, the actual accusations are false. This submission was an earlier version of pgrust 0.4 called pgrust 0.4-preview. We just open-sourced pgrust 0.4-beta yesterday, and you can fully audit it here[0]. Rather than contest the accusations on GitHub, we figured it would be easier to complete pgrust 0.4 and open-source it to show that we're being honest. You can also verify what I'm saying yourself against the 0.4-preview binary we submitted to ClickBench here[1]. I don't have a way to prove this is the exact binary submitted, but if ClickHouse provided the SHA of the binary they tested against, it would match.

    The three main accusations boil down to: 1) We used handwritten code optimized against each query in the benchmark, 2) We cached the results between query runs, and 3) We didn't support queries outside of the 43 ClickBench queries.

    First, we did not use handwritten queries. The code Alexey pointed out does not run during the benchmark. The design of 0.4 relies on a JIT compiler that compiles your SQL to a Lisp-based intermediate representation that we call Gravity IR that we then convert to x86 and ARM assembly. While developing 0.4, I wrote handwritten code against the IR for benchmarking purposes. The idea was that if queries written in our IR are fast, and we can compile good enough IR fast, then our new query engine will be fast. This is also the reason the directory those "kernels" are in is called "irprobe" - it was a probe of our IR to make sure it was fast. Our 0.4-preview submission went through the pipeline: SQL query -> Gravity IR -> assembly.

    Second, the "result-cache" Alexey observed is the result of a cache we use to cache which predicates match which strings in our dictionaries we use for dictionary compression. Alexey wasn't aware of this cache and assumed we were just caching the query results. On ClickBench you are allowed to use caches like this that hold intermediate data[2]. ClickHouse has its own cache similar to this called the Condition Cache[3]. In our newest submission, we were upfront about all the caching we do and how to disable each cache[4].

    The third claim that we didn't support other queries outside of the 43 ClickBench queries is partially false, but does have some ground to it. As I said before, pgrust 0.4-preview was an early version of pgrust 0.4. Once our new JIT compiler supported enough SQL to handle the ClickBench queries, we decided to submit it to ClickBench. AFAIK, ClickBench has no rules about how much SQL you need to support. On top of that, we did ship that we did ship 0.4-preview with our old query engine. It would have been a two-line change to fall back to it and support a much broader set of queries outside what our new engine could support. Those queries would still be fast, but not quite as fast as our new engine was. Our new 0.4-beta release does now support all Postgres SQL (minus some bugs around CTEs and MERGE)

    To sum up, pgrust 0.4-preview was an early build of pgrust 0.4 while it was in development. It was a real query engine and actually executed queries. I acknowledge that we submitted it prematurely, when it supported a narrow set of SQL, and we should have ensured broader SQL coverage before submitting it. With our new submission that solves these problems, we're hoping for pgrust to be reinstated. If there's anything off about the submission, please point it out and we're happy to fix it.

    [0] https://github.com/malisper/pgrust/releases/tag/v0.4-beta

    [1] https://pgrust.com/downloads/archive/2026-09-clickbench-snap...

    [2] https://github.com/ClickHouse/ClickBench/blob/main/README.md

    [3] https://clickhouse.com/docs/concepts/features/performance/ca...

    [4] https://github.com/ClickHouse/ClickBench/pull/2751

    • jimmydoe 6 minutes ago
      thanks for the explanation, let's first see if the binary matches.
    • philipwhiuk 6 minutes ago
      Can you explain "PGRUST_GRAVITY_Q20_VARIANT"?
      • malisper 1 minute ago
        It's a feature flag we used for our own internal benchmarking. It doesn't get hit when you run SQL.