I add a loger tocheck the return status, I am not sure if it is enough, 
e.g, in case of request failure, should a action perform?
  UpdateResponse resp = solrServer.add(inputDoc);
             if(resp.getStatus()==0){
                 logger.debug("update request was successful for pid: 
{}", pid);
                 solrServer.commit();
             }
             else{logger.debug("update request has error, code: {} for 
pid: {}", resp.getStatus(),pid);}


On 08.10.2013 16:13, Andrew Woods wrote:
>
> In 
> fcrepo-jms-indexer-core/src/main/java/org/fcrepo/indexer/SolrIndexer.java:
>
> > +            UpdateResponse resp = solrServer.add(inputDoc);
> > +            solrServer.commit();
> > +        } catch (SolrServerException e) {
> > +            throw new IOException(e);
> > +        }
> > +    }
> > +
> > +    /**
> > +     * Implementation of the remove method overriding org.fcrepo.indexer.Indexer
> > +     * for the Solr indexer implementation
> > +     * @see org.fcrepo.indexer.Indexer#remove(java.lang.String)
> > +     */
> > +    @Override
> > +    public void remove(String pid) throws IOException {
> > +        try {
> > +            solrServer.deleteById(pid);
>
> Likewise, it may be useful to check the returned UpdateResponse.
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/awoods/fcrepo-jms-indexer-pluggable/pull/1/files#r6826536>.
>