We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Problem description The FlinkPravegaReader has a test FlinkPravegaReaderTest that is best understood as an integration test. Coverage is too low.
FlinkPravegaReader
FlinkPravegaReaderTest
Suggestions for an improvement
FlinkPravegaReaderITCase
EventStreamReader
The text was updated successfully, but these errors were encountered:
@EronWright, I would gladly take this issue.
Also, I assume that this together with #67 should subsume #51?
Sorry, something went wrong.
@tzulitai sounds great, would you mind following the style seen in #76 ?
Sure, no problem 👌
This is fixed
tzulitai
No branches or pull requests
Problem description
The
FlinkPravegaReader
has a testFlinkPravegaReaderTest
that is best understood as an integration test. Coverage is too low.Suggestions for an improvement
FlinkPravegaReader
toFlinkPravegaReaderITCase
.EventStreamReader
.The text was updated successfully, but these errors were encountered: