ctubbsii opened a new pull request #960: Use NewTableConfiguration (and also 
trivial code cleanup)
URL: https://github.com/apache/fluo/pull/960
 
 
   This PR consists of two commits (best left as two separate commits). The 
first cleans up some warnings, and formatting. The second makes use of 
NewTableConfiguration to create tables already pre-loaded with their initial 
properties at the time of creation.
   
   After switching to NewTableConfiguration, I was reliably able to reproduce 
the bug which was fixed in #959 and also subsequently able to confirm it was 
fixed after that was merged in. So, I think this using this feature has merit.
   
   This does require updating the minimum version of Accumulo to 1.7, which I 
think is fine, since 1.6 is EOL anyway. I also updated the Travis-CI config in 
the second commit so that tests are run in Travis with 1.8.1 routinely, since 
that's the current version of Accumulo.

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to