Skip to content

Should the arg of octet_length() be a bare Expr instead of Vec<Expr>? #10676

@yutannihilation

Description

@yutannihilation

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions