accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubbsii <...@git.apache.org>
Subject [GitHub] accumulo issue #282: Added version to the REST API
Date Mon, 24 Jul 2017 18:42:04 GMT
Github user ctubbsii commented on the issue:

    https://github.com/apache/accumulo/pull/282
  
    I think such a test really only verifies that Jackson is working properly.
    
    I suppose it's reasonable to write an integration test for the REST API, but it seems
to me that should be done in a comprehensive way, rather than as a series of one-offs for
specific fields in specific endpoints. Could be a follow on issue. Will create a JIRA for
it, so it can be done as follow-on, but I think it's fine to merge this in now.


---
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