Describe the bug
This is a minor issue about consistency. Currently, the signature of octet_length() is this, but I think this function takes only one argument just like bit_length().
pub fn octet_length(args: Vec<Expr>) -> Expr
https://docs.rs/datafusion/38.0.0/datafusion/functions/string/expr_fn/fn.octet_length.html
To Reproduce
n/a
Expected behavior
n/a
Additional context
n/a