[
https://issues.apache.org/jira/browse/BEAM-7770?focusedWorklogId=437757&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-437757
]
ASF GitHub Bot logged work on BEAM-7770:
----------------------------------------
Author: ASF GitHub Bot
Created on: 27/May/20 12:05
Start Date: 27/May/20 12:05
Worklog Time Spent: 10m
Work Description: aromanenko-dev commented on pull request #11357:
URL: https://github.com/apache/beam/pull/11357#issuecomment-634615002
@iemejia You are right - it's clear that `readAll()` is tested implicitly
through calling `read()`. However, tests are not (and shouldn't be!) aware
about internal implementation of tested method, so we have to have tests for
all public API methods. So, yes, please add an extra test for that.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
Issue Time Tracking
-------------------
Worklog Id: (was: 437757)
Time Spent: 1h (was: 50m)
> Add ReadAll transform for SolrIO
> --------------------------------
>
> Key: BEAM-7770
> URL: https://issues.apache.org/jira/browse/BEAM-7770
> Project: Beam
> Issue Type: Improvement
> Components: io-java-solr
> Reporter: Ismaël Mejía
> Assignee: Ismaël Mejía
> Priority: P3
> Time Spent: 1h
> Remaining Estimate: 0h
>
> SolrIO already uses internally a composable approach but we need to expose an
> explicit ReadAll transform that allows user to create reads in the middle of
> the Pipeline to improve composability (e.g. Reads in the middle of a
> Pipeline).
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
