perf: Reserve space for holders_ in DecodedArgs constructor (#15418)

Summary:
Pull Request resolved: #15418

The DecodedArgs constructor was populating the holders_ vector without
pre-allocating space, which could cause multiple reallocations as the
vector grows. This adds a reserve() call before the loop to allocate
space upfront for all elements, improving performance by avoiding
unnecessary allocations and copies.

Reviewed By: Yuhta

Differential Revision: D86366246

fbshipit-source-id: ded985a5a2410b578a97626d2f6e7d88654750eb

Author: ttsugriy

velox/expression/DecodedArgs.h

@@ -36,6 +36,7 @@ class DecodedArgs {
       const SelectivityVector& rows,
       const std::vector<VectorPtr>& args,
       exec::EvalCtx& context) {
+    holders_.reserve(args.size());
     for (auto& arg : args) {
       holders_.emplace_back(context, *arg, rows);
     }