ColtenOuO commented on issue #70503: URL: https://github.com/apache/airflow/issues/70503#issuecomment-5090084999
Hi @shahar1 , sorry for the direct ping, and no rush at all. Since #70508 no reviewer is assigned yet, would you have time to take a look whenever it suits you? I'd mainly like to confirm the approach is what you had in mind - in particular, I moved **both** provision checks rather than only `cypher is None`: after #70373, `Neo4jOperator(sql=...)` is valid with `cypher` unset, so the constructor check has to be `cypher is None and sql is None`, and splitting it from the mutual-exclusivity check felt arbitrary once written that way. If the approach looks right, I'd be glad to take `databricks_repos.py`, `agent.py` and `document_loader.py` as well. -- 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]
