Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Threaded View
Accumulo >> mail # dev >> Adding windows-specific profiles to exclude unix-specific tests.

Copy link to this message
Re: Adding windows-specific profiles to exclude unix-specific tests.
I do not think we should maintain separate profiles. If we were to
vote on that, I'd vote against for the following reasons:

1) It adds build complexity and is difficult to maintain
2) Restricting tests like this has the potential to mask serious bugs
3) It doesn't address the underlying compatibility problems

As an alternative, I'd rather see the code (or the tests) improved so
they function better in Windows. If it means that some code needs to
function differently on one environment than another, then the code
should do that check itself and behave accordingly. I don't see why
some blocks of code can't have a check for the OS, when absolutely
necessary (I suspect often it isn't necessary... there's probably more
platform-independent ways of doing what is failing):

if (System.getProperty("os.name").toLowerCase().contains("windows")) {
  // do stuff on windows
} else {
  // do other stuff
Christopher L Tubbs II
On Fri, Jun 14, 2013 at 9:59 PM, David Medinets
> In March there was little enthusiasm for this change. However, since then
> the MiniAccumulo cluster has been ported to Windows. If people want the MAC
> on Windows, then the whole system should be compilable ... if not
> executable.
> http://affy.blogspot.com/2013/03/running-accumulo-unit-tests-on-windows.htmllists
> the the non-working tests; at least those that did not work in March.
> Can we vote on this change? Most of the work has already been done. It
> seems harmless.