From 1318943197bbf4c19489b310ece54a86e891b232 Mon Sep 17 00:00:00 2001 From: Arun Sharma Date: Fri, 16 Oct 2026 13:27:05 -0700 Subject: [PATCH] fix(algo): add template disambiguator for macOS Clang build auto-deduced dataVector in GenericEmbeddingVertexCompute is a dependent type under Clang, which rejects dataVector->setValue without the template keyword. GCC accepts it, so Linux builds passed while macOS failed with: error: use 'template' keyword to treat 'setValue' as a dependent template name Add the template keyword; valid on both compilers. --- algo/src/include/function/gds_generic_algo.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/algo/src/include/function/gds_generic_algo.h b/algo/src/include/function/gds_generic_algo.h index 38bcffff..3f427e18 100644 --- a/algo/src/include/function/gds_generic_algo.h +++ b/algo/src/include/function/gds_generic_algo.h @@ -263,9 +263,9 @@ class GenericEmbeddingVertexCompute final : public function::GDSResultVertexComp auto* dataVector = common::ListVector::getDataVector(embVector.get()); for (auto k = 0u; k < emb.size(); ++k) { if constexpr (std::is_same_v) { - dataVector->setValue(entry.offset + k, emb[k]); + dataVector->template setValue(entry.offset + k, emb[k]); } else { - dataVector->setValue(entry.offset + k, emb[k]); + dataVector->template setValue(entry.offset + k, emb[k]); } } localFT->append(vectors);