[FEATURE]: remove limitation: notInArray requires at least one value

[sys13]

Describe what you want

notInArray requires at least one value. Would like to allow for empty arrays

[DenisBessa]

This would be nice. It leads to unnecessary runtime errors.

[Angelelz]

Can you share a code snippet? What is causing the error?

[DenisBessa]

The behavior of inArray and notInArray is the same.

When providing an empty array as argument, it leads to a runtime error.

  const someEmptyArray = []
  const thisWillGenerateAnRuntimeError = await db.select().from(customers).where(inArray(customers.id, someEmptyArray));

  const someNonEmptyArray = [1, 2, 3]
  const thisIsOk = await db.select().from(customers).where(inArray(customers.id, someNonEmptyArray));

[Angelelz]

How do you think drizzle should handle an empty array? just remove the filter altogether?
I think the team discussed this briefly but I don't remember a decision. I guess drizzle could just return false if inArray is passed an empty array. In any case, I'll close this issue in favor of #1295. That one already has a couple reactions. Let's continue the discussion over there.