[
https://issues.apache.org/jira/browse/BEAM-3848?focusedWorklogId=84550&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-84550
]
ASF GitHub Bot logged work on BEAM-3848:
----------------------------------------
Author: ASF GitHub Bot
Created on: 26/Mar/18 20:31
Start Date: 26/Mar/18 20:31
Worklog Time Spent: 10m
Work Description: iemejia commented on a change in pull request #4905:
[BEAM-3848] Enables ability to retry Solr writes on error (SolrIO)
URL: https://github.com/apache/beam/pull/4905#discussion_r177224620
##########
File path:
sdks/java/io/solr/src/main/java/org/apache/beam/sdk/io/solr/SolrIO.java
##########
@@ -661,25 +746,51 @@ public void processElement(ProcessContext context)
throws Exception {
SolrInputDocument document = context.element();
batch.add(document);
if (batch.size() >= spec.getMaxBatchSize()) {
- flushBatch();
+ flushBatch(solrClient, batch);
Review comment:
Maybe if you change the tests as suggested below to not test the Fn directly
you may avoid passing solrClient at all.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 84550)
Time Spent: 2h 20m (was: 2h 10m)
> SolrIO: Improve retrying mechanism in client writes
> ---------------------------------------------------
>
> Key: BEAM-3848
> URL: https://issues.apache.org/jira/browse/BEAM-3848
> Project: Beam
> Issue Type: Improvement
> Components: io-java-solr
> Affects Versions: 2.2.0, 2.3.0
> Reporter: Tim Robertson
> Assignee: Tim Robertson
> Priority: Minor
> Time Spent: 2h 20m
> Remaining Estimate: 0h
>
> A busy SOLR server is prone to return RemoteSOLRException on writing which
> currently fails a complete task (e.g. a partition of a spark RDD being
> written to SOLR).
> A good addition would be the ability to provide a retrying mechanism for the
> batch in flight, rather than failing fast, which will most likely trigger a
> much larger retry of more writes.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
