Change Resource dunder hash to reduce collisions

dgetu · dgetu · commit 32ce84880e40 · 2021-05-25T11:12:31.000-07:00
diff --git a/opentelemetry-sdk/src/opentelemetry/sdk/resources/__init__.py b/opentelemetry-sdk/src/opentelemetry/sdk/resources/__init__.py
@@ -234,7 +234,9 @@ def __eq__(self, other: object) -> bool:
         )
 
     def __hash__(self):
-        return hash(dumps(self._attributes, sort_keys=True) + self._schema_url)
+        return hash(dumps(self._attributes, sort_keys=True)) + 31 * hash(
+            self._schema_url
+        )
 
 
 _EMPTY_RESOURCE = Resource({}, "")

Original file line number	Diff line number	Diff line change
`@@ -234,7 +234,9 @@ def __eq__(self, other: object) -> bool:`
`234`	`234`	`)`
`235`	`235`
`236`	`236`	`def __hash__(self):`
`237`		`- return hash(dumps(self._attributes, sort_keys=True) + self._schema_url)`
	`237`	`+ return hash(dumps(self._attributes, sort_keys=True)) + 31 * hash(`
	`238`	`+ self._schema_url`
	`239`	`+ )`
`238`	`240`
`239`	`241`
`240`	`242`	`_EMPTY_RESOURCE = Resource({}, "")`