Skip to content
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

add batchingOnly convenience method #110

Merged
1 commit merged into from
Mar 10, 2017

Conversation

AlecZorab
Copy link

  • similarly to batchingNotSupported, but for sources that only do
    batching (or for which batching is just as natural)

- similarly to batchingNotSupported, but for sources that only do
batching (or for which batching is just as natural)
@AlecZorab AlecZorab mentioned this pull request Mar 9, 2017
@codecov-io
Copy link

Codecov Report

Merging #110 into master will decrease coverage by -0.33%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #110      +/-   ##
==========================================
- Coverage   76.79%   76.47%   -0.33%     
==========================================
  Files          10       10              
  Lines         237      238       +1     
  Branches        2        2              
==========================================
  Hits          182      182              
- Misses         55       56       +1
Impacted Files Coverage Δ
shared/src/main/scala/datasource.scala 66.66% <0%> (-13.34%)

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 b7ee4a1...dd934c4. Read the comment docs.

@ghost
Copy link

ghost commented Mar 10, 2017

Nice, thanks for taking the time to contribute!

@ghost ghost self-assigned this Mar 10, 2017
@ghost ghost merged commit 68a5a1f into xebia-functional:master Mar 10, 2017
This pull request was closed.
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.

None yet

2 participants