potiuk commented on issue #18967:
URL: https://github.com/apache/airflow/issues/18967#issuecomment-988627777


   
   > For all Oracle Database's versions you can replace "select 1" with "select 
1 from dual"
   > 
   > dual is a logical table present in every oracle schema you're connecting 
to.
   > 
   > Probably is a simpler implementation of the test.
   
   Surely for Oracle yes so this mean that you have to add oracle specific 
'test_connection" method in 
https://github.com/apache/airflow/blob/main/airflow/providers/oracle/hooks/oracle.py
 to override "test_connection" in 
https://github.com/apache/airflow/blob/main/airflow/hooks/dbapi.py#L376 
   
   Would you @hkjang @gbonazzoli care to make a PR fixing it for Oracle Hook? 
Sounds super easy and since you already use Oracle it wll be easy for you to 
test the fix.
   


-- 
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]


Reply via email to