Skip to content

Conversation

@afedulov
Copy link
Contributor

This is a trivial docs fix.

@flinkbot
Copy link
Collaborator

flinkbot commented Sep 21, 2022

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@afedulov afedulov force-pushed the hotfix-kinesis-docs branch 2 times, most recently from 2713b8e to 9cd7552 Compare September 21, 2022 22:47
@afedulov afedulov changed the title [hotfix][docs] Fix typo in Kinesis Connector docs. [hotfix][docs] Fix typos in Kinesis Connector docs. Sep 21, 2022
This is the preferred strategy for the majority of applications.
However, jobs with parallelism greater than 1 will result in tasks competing to register and acquire the stream consumer ARN.
For jobs with very large parallelism this can result in an increased start-up time.
The describe operation has a limit of 20 [transactions per second](https://docs.aws.amazon.com/kinesis/latest/APIReference/API_DescribeStreamConsumer.html),
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this is intentional, the API is called DescribeStreamConsumer, could be more explicit and change to

The DescribeStreamConsumer operation

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, ok, that makes sense, updated accordingly.

@dannycranmer dannycranmer merged commit c0d9007 into apache:master Sep 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants