Impl spark bit not function

[kazantsev-maksim]

Which issue does this PR close?

• Closes [datafusion-spark] Implement Spark bitwise_not function #17150.
  Part of [EPIC] Complete datafusion-spark Spark Compatible Functions #15914.

Rationale for this change

What changes are included in this PR?

Implement spark bit_not function

Are these changes tested?

Are there any user-facing changes?

Yes, new function on spark crate

[comphead]

Thanks @kazantsev-maksim btw I cannot find a bit_not Spark function in https://spark.apache.org/docs/latest/api/sql/

[kazantsev-maksim]

Thanks @kazantsev-maksim btw I cannot find a bit_not Spark function in https://spark.apache.org/docs/latest/api/sql/

Hmm, i find in source code: https://github.com/apache/spark/blob/418cf56ed4dc440b36ebab37d2aac898278215a8/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/bitwiseExpressions.scala#L178

[comphead]

Right https://spark.apache.org/docs/latest/api/sql/#_22
please add .slt test

[Jefffrey]

I'll pull in my comments from the previous PR (#17155) as it seems it was closed and reopened here:

• Should the function be named bitwise_not to be consistent with the (Py)Spark function?
  • https://spark.apache.org/docs/latest/api/python/reference/pyspark.sql/api/pyspark.sql.functions.bitwise_not.html
• Would benefit from having some SLT tests

[kazantsev-maksim]

I'll pull in my comments from the previous PR (#17155) as it seems it was closed and reopened here:

• Should the function be named bitwise_not to be consistent with the (Py)Spark function?

  • https://spark.apache.org/docs/latest/api/python/reference/pyspark.sql/api/pyspark.sql.functions.bitwise_not.html

• Would benefit from having some SLT tests

Thanks @Jefffrey, renamed func to bitwise_not

[Jefffrey]

Could consider adding a roundtrip test (e.g. SELECT bitwise_not(bitwise_not(0)))

But other than the CI failure (looks like fmt issue) this PR looks good to go 👍

[Jefffrey]

Thanks @kazantsev-maksim