akshaychitneni commented on code in PR #34840: URL: https://github.com/apache/airflow/pull/34840#discussion_r1385350060
########## airflow/providers/papermill/operators/papermill.py: ########## @@ -17,17 +17,102 @@ # under the License. from __future__ import annotations +from functools import cached_property from typing import TYPE_CHECKING, ClassVar, Collection, Sequence import attr +from papermill.utils import remove_args, merge_kwargs +from pydantic import typing + import papermill as pm +from papermill.engines import NBClientEngine +from papermill.clientwrap import PapermillNotebookClient +from jupyter_client.manager import AsyncKernelManager +from jupyter_client.client import KernelClient +from traitlets import Unicode from airflow.lineage.entities import File from airflow.models import BaseOperator +from airflow.providers.papermill.hooks.kernel import KernelHook if TYPE_CHECKING: from airflow.utils.context import Context +REMOTE_KERNEL_ENGINE = "remote_kernel_engine" Review Comment: We are registering a remote kernel engine(custom papermill engine) so the operator can work with remote kernels. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org