Fix generate_random_uuid bug

[andyfengHKU]

generate_random_uuid is a random function so we need to generate a new random value for each tuple. In such case, we cannot apply factorization technique to reduce duplicate intermediate result.

Our solution is to fall back to tuple at a time evaluation for random function.

[codecov]

Codecov Report

Attention: Patch coverage is 93.93939% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 93.50%. Comparing base (f2619d5) to head (53e0d25).

Files	Patch %	Lines
src/include/function/pointer_function_executor.h	50.00%	2 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #2941   +/-   ##
=======================================
  Coverage   93.50%   93.50%           
=======================================
  Files        1121     1121           
  Lines       42859    42881   +22     
=======================================
+ Hits        40075    40096   +21     
- Misses       2784     2785    +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[acquamarin]

Can you update the GenRandomUUID operation as well?

struct GenRandomUUID {
    static void operation(common::ku_uuid_t& input, void* dataPtr) {
        input = common::UUID::generateRandomUUID(
            reinterpret_cast<main::ClientContext*>(dataPtr)->getRandomEngine());
    }
};

And executor:

     template<typename RESULT_TYPE, typename OP>
     static void execute(common::ValueVector& result, void* dataPtr) {
+        if (result.state->selVector->isUnfiltered()) {
+            for (auto i = 0u; i < result.state->selVector->selectedSize; i++) {
+                OP::operation(result.getValue<RESULT_TYPE>(i), dataPtr);
+            }
+        } else {
+            for (auto i = 0u; i < result.state->selVector->selectedSize; i++) {
+                auto pos = result.state->selVector->selectedPositions[i];
+                OP::operation(result.getValue<RESULT_TYPE>(pos), dataPtr);
+            }
+        }
     }