SIZE vs LEN (hashtag2796)

[vikramsubramanian]

In order to get a list length, the document suggests using [len]( which throws a runtime error. The working name of a function with the functionality is SIZE. Likely just a documentation issue? )

[mayil-ai[bot]]

Summary: Documentation suggests using "len" to get list length, but the correct function name is "SIZE".

Possible Solution

Based on the provided information, here is the solution to the issue:

• Update the documentation to replace any incorrect references to len with SIZE.
• Search for instances where len is used in the codebase and replace them with SIZE.
• Verify that the SIZE function is correctly implemented and handles all expected input types.
• Add or update unit tests to cover the usage of SIZE and ensure no runtime errors occur.
• Ensure that the SIZE function is registered and available in the function set where list operations are defined, as shown in the vector_list_functions.cpp snippet.
• If len is a member variable (as seen in linenoise.cpp and list_len_function.h), ensure it is accessed correctly and not confused with a function call.
• Review the error log related to the runtime error when using len to identify any specific cases or edge conditions that need to be handled.
• After making changes, run the full test suite to ensure that no new issues have been introduced and that the runtime error is resolved.

Code snippets to check

• src/include/function/list/functions/list_len_function.h

  This file contains the implementation of the list length function, which is likely to be the source of confusion between 'len' and 'size'.

• src/function/vector_list_functions.cpp

  This file defines the SIZE function for lists and strings, which is the correct function to use according to the issue description.

• src/include/common/enums/expression_type.h

  This file contains constants for function names, which might need to be updated if there is a discrepancy between the documentation and the actual function names.