The commits have been fixed accordingly. The extension of the DatabricksOperator may also be incorporated as well. However, the current objective is to extend the hook functionality leveraging the available api.
Thanks, Wayne On Tue, Aug 28, 2018 at 5:16 PM Andrew Chen <[email protected]> wrote: > Looks reasonable. I can take a review once the comments that @ashb > <https://github.com/ashb> brought up are addressed. Is the eventual goal > to extend these to DatabricksOperators eventually? > > — > You are receiving this because you authored the thread. > Reply to this email directly, view it on GitHub > <https://github.com/apache/incubator-airflow/pull/3817#issuecomment-416743238>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AHlcZrhpAehd0ABq2cKFiHqysSiw7SX0ks5uVbMQgaJpZM4WP8OR> > . > -- With kind regards, Wayne Morris [ Full content available at: https://github.com/apache/incubator-airflow/pull/3817 ] This message was relayed via gitbox.apache.org for [email protected]
