1fanwang commented on code in PR #72173:
URL: https://github.com/apache/airflow/pull/72173#discussion_r3891837691
##########
providers/apache/iceberg/src/airflow/providers/apache/iceberg/triggers/iceberg.py:
##########
@@ -132,7 +132,7 @@ async def run(self) -> AsyncIterator[TriggerEvent]:
store = getattr(self, "asset_state_store", None)
if store is not None:
try:
- stored = await asyncio.to_thread(store.get, WATERMARK_KEY)
+ stored = await store.aget(WATERMARK_KEY)
Review Comment:
@jason810496 done in https://github.com/apache/airflow/pull/72312. It awaits
`aget`/`aset` where the store has them and otherwise runs the blocking accessor
through `asyncio.to_thread`, so the watcher works on 3.3.0 and 3.3.1 and picks
up the async accessors automatically once
https://github.com/apache/airflow/pull/72127 lands.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
