You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2018/12/07 16:47:03 UTC

[GitHub] mikewalch commented on issue #808: Fixes #788 - Remove ClientInfo from API

mikewalch commented on issue #808: Fixes #788 - Remove ClientInfo from API
URL: https://github.com/apache/accumulo/pull/808#issuecomment-445292637
 
 
   Tests are failing for this PR due to Apilyzer:
   
   ```
   Problems : 
   
     CONTEXT              TYPE                                                         FIELD/METHOD                        NON-PUBLIC REFERENCE
   
     METHOD_PARAM         org.apache.accumulo.core.client.mapreduce.AbstractInputFormat setClientInfo(...)                  org.apache.accumulo.core.clientImpl.ClientInfo
     METHOD_RETURN        org.apache.accumulo.core.client.mapreduce.AbstractInputFormat getClientInfo(...)                  org.apache.accumulo.core.clientImpl.ClientInfo
     METHOD_PARAM         org.apache.accumulo.core.client.mapreduce.AccumuloOutputFormat setClientInfo(...)                  org.apache.accumulo.core.clientImpl.ClientInfo
     METHOD_RETURN        org.apache.accumulo.core.client.mapreduce.AccumuloOutputFormat getClientInfo(...)                  org.apache.accumulo.core.clientImpl.ClientInfo
     METHOD_PARAM         org.apache.accumulo.core.client.mapred.AbstractInputFormat   setClientInfo(...)                  org.apache.accumulo.core.clientImpl.ClientInfo
     METHOD_RETURN        org.apache.accumulo.core.client.mapred.AbstractInputFormat   getClientInfo(...)                  org.apache.accumulo.core.clientImpl.ClientInfo
     METHOD_PARAM         org.apache.accumulo.core.client.mapred.AccumuloOutputFormat  setClientInfo(...)                  org.apache.accumulo.core.clientImpl.ClientInfo
     METHOD_RETURN        org.apache.accumulo.core.client.mapred.AccumuloOutputFormat  getClientInfo(...)                  org.apache.accumulo.core.clientImpl.ClientInfo
   ```
   
   ClientInfo need to be removed from M/R API before this can merged.  So this PR is dependent on #804

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services