[GitHub] ashb commented on a change in pull request #4129: [AIRFLOW-3294] Update connections form and integration docs

[GitBox <gi...@apache.org>]

ashb commented on a change in pull request #4129: [AIRFLOW-3294] Update connections form and integration docs
URL: https://github.com/apache/incubator-airflow/pull/4129#discussion_r230731775
 
 

 ##########
 File path: docs/integration.rst
 ##########
 @@ -1011,3 +1011,13 @@ QuboleFileSensor
 ''''''''''''''''
 
 .. autoclass:: airflow.contrib.sensors.qubole_sensor.QuboleFileSensor
+
+QuboleCheckOperator
+'''''''''''''''''''
+
+.. autoclass:: airflow.contrib.operators.qubole_check_operator.QuboleCheckOperator
+
+QuboleValueCheckOperator
+''''''''''''''''''''''''
+
+.. autoclass:: airflow.contrib.operators.qubole_check_operator.QuboleValueCheckOperator
 
 Review comment:
   We need to work out what we put in this section more generally.
   
   In the first case _all_ operators/hooks/sensors should be in the `docs/code.rst` as that is the complete list.
   
   Secondly I'm not sure that listing operators in here is all that useful - I think this doc should be limited to interesting integrations, i.e. ones that need settings or configuring (like the reverse proxy) rather than just A second list of classes with no further description of them.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services