-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[AIRFLOW-XXX] Add How-To-Guide to GCP PubSub #401
Conversation
@@ -572,6 +588,10 @@ def execute(self, context): | |||
class PubSubPublishOperator(BaseOperator): | |||
"""Publish messages to a PubSub topic. | |||
|
|||
.. seealso:: | |||
For more information on how to use this operator, take a look at the guide: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am not sure about the comma, @mschickensoup WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sentence is used widely with comma 🤔 . Is it proper sentence @mschickensoup ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's correct, feel free to leave it as it is :-) -> https://developers.google.com/style/cross-references?hl=vi
Codecov Report
@@ Coverage Diff @@
## master #401 +/- ##
=========================================
Coverage ? 83.47%
=========================================
Files ? 632
Lines ? 36644
Branches ? 0
=========================================
Hits ? 30588
Misses ? 6056
Partials ? 0
Continue to review full report at Codecov.
|
44dfc35
to
cf755e0
Compare
PR to apache created: apache#6497 |
cf755e0
to
4e31868
Compare
4e31868
to
b1ad0d1
Compare
on Google Cloud Platform or elsewhere on the Internet. | ||
|
||
Publisher applications can send messages to a topic and other applications can subscribe to that topic to receive the messages. | ||
By decoupling senders and receivers, Google Cloud PubSub allows developers to communicate between independently written applications. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
By decoupling senders and receivers, Google Cloud PubSub allows developers to communicate between independently written applications. | |
By decoupling senders and receivers Google Cloud PubSub allows developers to communicate between independently written applications. |
Creating a PubSub subscription | ||
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | ||
|
||
The subscription is named resource representing the stream of messages from a single, specific topic, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this subscription any kind of subscription or you're referring to the specific one?
Deleting a PubSub subscription | ||
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ | ||
|
||
The :class:`~airflow.providers.google.cloud.operators.pubsub.PubSubSubscriptionDeleteOperator` operator would delete a subscription. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The :class:`~airflow.providers.google.cloud.operators.pubsub.PubSubSubscriptionDeleteOperator` operator would delete a subscription. | |
The :class:`~airflow.providers.google.cloud.operators.pubsub.PubSubSubscriptionDeleteOperator` operator deletes the subscription. |
Please, don't use would
:3 BTW, are you again referring to the specific subscription in lights of the class you mentioned?
Deleting a PubSub topic | ||
^^^^^^^^^^^^^^^^^^^^^^^ | ||
|
||
The :class:`~airflow.providers.google.cloud.operators.pubsub.PubSubTopicDeleteOperator` operator would delete a topic. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The :class:`~airflow.providers.google.cloud.operators.pubsub.PubSubTopicDeleteOperator` operator would delete a topic. | |
The :class:`~airflow.providers.google.cloud.operators.pubsub.PubSubTopicDeleteOperator` operator deletes a topic. |
No would
pls.
Make sure you have checked all steps below.
Jira
Description
Tests
Commits
Documentation