accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From keith-turner <...@git.apache.org>
Subject [GitHub] accumulo pull request: [ACCUMULO-4147] - Add Accumulo Namespace Op...
Date Fri, 26 Feb 2016 16:08:04 GMT
Github user keith-turner commented on the pull request:

    https://github.com/apache/accumulo/pull/74#issuecomment-189345114
  
    > Any thoughts on where I should look?
    
    Looking at the test I have a strong suspicion that the reason the test is failing is because
its using MockAccumulo.   When things worked w/ your Python test client, was that proxy back
a mini or real accumulo instance?
    
    I would discourage using Mock.  Its been deperecated because it was not being well maintained
as features (like namspaces) were added to Accumulo.  I would also encourage you to add your
namespace test to SimpleProxyBase.  TBinaryProxyIT, TJsonProtocolProxyIT, etc all extend SimpleProxyBase
but run configure a different protocol.  All of the test in SimpleProxyBase are run using
multiple thrift protocols.   Also, the test in SimpleProxyBase use MiniAccumulo instead of
MockAccumulo.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message