Skip to content

fix: Better error message when subscription path is invalid.#87

Merged
jiangmichaellll merged 2 commits into
masterfrom
jiangmichael-explicit-sub-path-error-msg
Feb 22, 2021
Merged

fix: Better error message when subscription path is invalid.#87
jiangmichaellll merged 2 commits into
masterfrom
jiangmichael-explicit-sub-path-error-msg

Conversation

@jiangmichaellll
Copy link
Copy Markdown
Contributor

No description provided.

@jiangmichaellll jiangmichaellll requested a review from a team February 22, 2021 20:58
@google-cla google-cla Bot added the api: pubsublite Issues related to the googleapis/java-pubsublite-spark API. label Feb 22, 2021
@product-auto-label product-auto-label Bot added the api: pubsublite Issues related to the googleapis/java-pubsublite-spark API. label Feb 22, 2021
@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 22, 2021

Codecov Report

Merging #87 (9561a01) into master (05e293f) will increase coverage by 1.15%.
The diff coverage is 50.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #87      +/-   ##
============================================
+ Coverage     58.31%   59.46%   +1.15%     
- Complexity       81       82       +1     
============================================
  Files            17       17              
  Lines           523      528       +5     
  Branches         17       18       +1     
============================================
+ Hits            305      314       +9     
+ Misses          215      210       -5     
- Partials          3        4       +1     
Impacted Files Coverage Δ Complexity Δ
...e/cloud/pubsublite/spark/PslDataSourceOptions.java 19.56% <50.00%> (+8.20%) 3.00 <0.00> (+1.00)
...m/google/cloud/pubsublite/spark/PslSparkUtils.java 95.71% <0.00%> (+0.06%) 14.00% <0.00%> (ø%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 05e293f...441fcc8. Read the comment docs.

Copy link
Copy Markdown
Contributor

@anguillanneuf anguillanneuf left a comment

Choose a reason for hiding this comment

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

Some lines are not covered by tests?

@jiangmichaellll
Copy link
Copy Markdown
Contributor Author

it's now all covered.

@jiangmichaellll jiangmichaellll merged commit 103e1eb into master Feb 22, 2021
@jiangmichaellll jiangmichaellll deleted the jiangmichael-explicit-sub-path-error-msg branch February 22, 2021 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: pubsublite Issues related to the googleapis/java-pubsublite-spark API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants