vincbeck commented on code in PR #30541:
URL: https://github.com/apache/airflow/pull/30541#discussion_r1162963866
##########
docs/apache-airflow-providers-amazon/transfer/dynamodb_to_s3.rst:
##########
@@ -25,7 +25,7 @@ to an existing Amazon Simple Storage Service (S3) bucket.
Prerequisite Tasks
------------------
-.. include:: ../_partials/prerequisite_tasks.rst
+.. include:: ../operators/_partials/prerequisite_tasks.rst
Review Comment:
`prerequisite_tasks.rst` refers to some instructions/help to use all
operators/sensors in amazon provider package. As such, we should move it to
`docs/apache-airflow-providers-amazon/_partials`
##########
docs/apache-airflow-providers-amazon/operators/redshift_cluster.rst:
##########
@@ -15,9 +15,9 @@
specific language governing permissions and limitations
under the License.
-===============
-Amazon Redshift
-===============
+=======================
+Amazon Redshift Cluster
Review Comment:
I would put `Cluster` in parenthesis. Otherwise users might think the whole
service name is `Amazon Redshift Cluster`
```suggestion
Amazon Redshift (Cluster)
```
##########
docs/apache-airflow-providers-amazon/operators/redshift_sql.rst:
##########
@@ -17,9 +17,9 @@
.. _howto/operator:RedshiftSQLOperator:
-===============
-Amazon Redshift
-===============
+===================
+Amazon Redshift SQL
Review Comment:
```suggestion
Amazon Redshift (SQL)
```
--
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]
