> On May 9, 2014, 5:46 p.m., Sean Busbey wrote: > > I don't see any tests to cover the non-blocking server. Could we add > > something? > > Christopher Tubbs wrote: > We already have comprehensive test coverage of the thrift server, from > all of our RPC calls. What additional coverage do you think is needed? For my > purposes, I think a test that fails if the Thrift version is ever switched to > something other than 0.9.1 would be sufficient, to ensure we revisit this. > > Sean Busbey wrote: > I'm just looking to make sure we have similar coverage to the original > class upstream. a note in the top level javadoc not to make changes and an > enforcer check that has a comment pointing to this class would be sufficient > for me.
Makes sense. I'll see if there's any upstream tests I could apply. If not, I'll just do the basic enforcer/javadoc changes. - Christopher ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21043/#review42612 ----------------------------------------------------------- On May 2, 2014, 7:15 p.m., Christopher Tubbs wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/21043/ > ----------------------------------------------------------- > > (Updated May 2, 2014, 7:15 p.m.) > > > Review request for accumulo, Sean Busbey and kturner. > > > Bugs: ACCUMULO-1691 > https://issues.apache.org/jira/browse/ACCUMULO-1691 > > > Repository: accumulo > > > Description > ------- > > Updates Thrift dependency to 0.9.1 with a hack to access a needed protected > field. > > > Diffs > ----- > > pom.xml 43aa5fb > > server/base/src/main/java/org/apache/accumulo/server/util/CustomNonBlockingServer.java > PRE-CREATION > server/base/src/main/java/org/apache/accumulo/server/util/TServerUtils.java > 6d9e4c7 > > Diff: https://reviews.apache.org/r/21043/diff/ > > > Testing > ------- > > mvn clean verify -Psunny > > > Thanks, > > Christopher Tubbs > >
