Skip to content

Commit e3fe8cd

Browse files
committed
tweak: drop _urn_aliases
It is not actually set anywhere and isn't in the substrait spec.
1 parent 8c9f6a2 commit e3fe8cd

File tree

1 file changed

+0
-5
lines changed

1 file changed

+0
-5
lines changed

src/substrait/extension_registry.py

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -257,8 +257,6 @@ def __init__(self, load_default_extensions=True) -> None:
257257
self._function_mapping: dict = defaultdict(dict)
258258
self._id_generator = itertools.count(1)
259259

260-
self._urn_aliases = {}
261-
262260
# Bidirectional URI <-> URN mapping (temporary during migration)
263261
self._uri_urn_bimap = UriUrnBiDiMap()
264262

@@ -330,8 +328,6 @@ def register_extension_dict(self, definitions: dict, uri: str) -> None:
330328
def lookup_function(
331329
self, urn: str, function_name: str, signature: tuple
332330
) -> Optional[tuple[FunctionEntry, Type]]:
333-
urn = self._urn_aliases.get(urn, urn)
334-
335331
if (
336332
urn not in self._function_mapping
337333
or function_name not in self._function_mapping[urn]
@@ -347,7 +343,6 @@ def lookup_function(
347343
return None
348344

349345
def lookup_urn(self, urn: str) -> Optional[int]:
350-
urn = self._urn_aliases.get(urn, urn)
351346
return self._urn_mapping.get(urn, None)
352347

353348
def lookup_uri_anchor(self, uri: str) -> Optional[int]:

0 commit comments

Comments
 (0)