[Dev] Fix CI failure caused by #7113

[Tishj]

GCC4.8 seems to not be happy about us wanting to implicitly construct a std::pair<string, BoundValueParameter> from a std::pair<string, Value> even though this constructor exists:

	explicit BoundParameterData(Value val) : value(std::move(val)), return_type(value.type()) {
	}

We now explicitly construct the pair, hopefully making GCC4.8 happy

This was the (seemingly innocuous) diff:

-		for (idx_t i = 0; i < bind_values.size(); i++) {
-			prepared_planner.parameter_data.emplace_back(bind_values[i]);
+		for (auto &pair : bind_values) {
+			prepared_planner.parameter_data.emplace(pair);

(they used to be vectors, and are now maps)