Kysely•5w ago

AggregateFunctionBuilder inside a SUM

Hi there, I'm trying to build

SUM(CASE WHEN IsNoShow = 0 AND IsDNA = 0 THEN 1 ELSE 0 END)

SUM(CASE WHEN IsNoShow = 0 AND IsDNA = 0 THEN 1 ELSE 0 END)

I'm having trouble producing the inner part of the SUM, because it thinks it is not legal. I would expect:

const rows = await db
  .selectFrom("person")
  .select((eb) =>
    eb(
      eb.lit(100),
      "*",
      eb.fn.sum<number>((eb) =>
        eb
          .case()
          .when(
            eb.and([
              eb("IsNoShow", "=", eb.lit(0)),
              eb("IsDNA", "=", eb.lit(0)),
            ]),
          )
          .then(1)
          .else(0)
          .end(),
      ),
    ).as("NegativeRate"),
  )
  .execute();

const rows = await db
  .selectFrom("person")
  .select((eb) =>
    eb(
      eb.lit(100),
      "*",
      eb.fn.sum<number>((eb) =>
        eb
          .case()
          .when(
            eb.and([
              eb("IsNoShow", "=", eb.lit(0)),
              eb("IsDNA", "=", eb.lit(0)),
            ]),
          )
          .then(1)
          .else(0)
          .end(),
      ),
    ).as("NegativeRate"),
  )
  .execute();

to work, but it doesn't:

Argument of type 'AggregateFunctionBuilder<Database, "person", number>' is not assignable to parameter of type 'OperandValueExpressionOrList<Database, "person", ExpressionWrapper<Database, "person", 100>>'.

Argument of type 'AggregateFunctionBuilder<Database, "person", number>' is not assignable to parameter of type 'OperandValueExpressionOrList<Database, "person", ExpressionWrapper<Database, "person", 100>>'.

dev link repro: https://kyse.link/ycYjL does anyone know how I can get this done? or is this unsupported?

Solution:

Here you go https://kyse.link/X1Vxc Our types are a bit too strict and give the type 100 instead of number for eb.lit(100)...

Jump to solution

3 Replies

Solution

koskimas•5w ago

Here you go https://kyse.link/X1Vxc Our types are a bit too strict and give the type 100 instead of number for eb.lit(100)

koskimas•5w ago

But you can just swap the multiplication around and there's no need to use eb.lit https://kyse.link/pnjEU This also works if you really want to use the literal there https://kyse.link/mSLWJ

OverHashOP•4w ago

Ahh nice, awesome! Thanks for explaining!

Gaming

Programming

AggregateFunctionBuilder inside a SUM

Did you find this page helpful?