opentelemetry_sdk: mistake in the docs?

[survived]

opentelemetry_sdk::Resource::new docs say that:

Values are de-duplicated by key, and the first key-value pair with a non-empty string value will be retained

However, looking at the code, it's not true: actually, the last key-value pair will be retained (no matter if value is empty or not):

opentelemetry-rust/opentelemetry-sdk/src/resource/mod.rs

Lines 80 to 92 in e4cba94

	pub fn new<T: IntoIterator<Item = KeyValue>>(kvs: T) -> Self {
	let mut attrs = HashMap::new();
	for kv in kvs {
	attrs.insert(kv.key, kv.value);
	}
	Resource {
	inner: Arc::new(ResourceInner {
	attrs,
	schema_url: None,
	}),
	}
	}