Changes to barrier violation code to correct some oversights and errors.
Merge branch 'master' of software.sandia.gov:/space/git/Trilinos
Stokhos:  Disabling KokkosArray kernels tests.

These tests use the older versions of the kokkos array kernels in the
research directory, some of which Carter has removed.
Tpetra: Distributor: Removed unused debugging code.

The code had been introduced to help debug deadlocks.  Introducing
message tags fixed the problem a while back, so I'm removing the
debugging code.

Build/Test Cases Summary
Enabled Packages: Tpetra, Belos, Ifpack2, Xpetra, MueLu
0) MPI_DEBUG => Test case MPI_DEBUG was not run! => Does not affect push readiness! (-1.00 min)
1) SERIAL_RELEASE => Test case SERIAL_RELEASE was not run! => Does not affect push readiness! (-1.00 min)
2) MPI_DEBUG_SS => passed: passed=282,notpassed=0 (4.97 min)
3) SERIAL_RELEASE_SS => passed: passed=260,notpassed=0 (4.36 min)
4) MPI_RELEASE_SS => passed: passed=282,notpassed=0 (7.20 min)
IOSS: Extract common function into utils class

(From Sierra)
IOSS: Use a stable sort instead of regular sort to maintain original order if possible

(From Sierra)
Use utils version of routine instead of internal; clean up field handling

(From Sierra)
EXODUS: Initialize structure so if there is an early return due to error, it contains somewhat reasonable values

(From Sierra)
SEACAS: conjoin - more informative error message

(From Sierra)
Revert "IOSS: Use a stable sort instead of regular sort to maintain original order if possible"

This reverts commit 27f287def1af70b3008b9f5398e1473c7e8d66a8.

(From Sierra)
IOSS: Maintain input block order better on output files

(From Sierra)

Build/Test Cases Summary
Enabled Packages: SEACASConjoin, SEACASExodus, SEACASIoss
Disabled Packages: Panzer,PyTrilinos
Enabled all Forward Packages
0) MPI_DEBUG => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
1) SERIAL_RELEASE => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
2) MPI_DEBUG_SS => passed: passed=120,notpassed=0 (27.32 min)
3) SERIAL_RELEASE_SS => passed: passed=122,notpassed=0 (28.65 min)
Other local commits for this build/test group: 16acd5a, 4a25f1a, 9b54c88, 2e535e1, 10c84ee, 3d7a56b
MueLu: more comments and disabled debugging stats

A form of the stats should eventually be enabled for a debug build.
They give details about how the diffusive partition assignment is progressing.
MueLu: Fixing bug in ArbitrateAndCommunicate.

One of this method's purposes is to provide tie-breaking among processors that doesn't depend on the PID.
It uses pseudo-random weights that are not supposed to be correlated across processors.  However, A&C was
using Tpetra::Vector::random(), which produces exactly the same random values locally across processors.
(To be fair, this is well-documented.)  So it was defaulting to using the highest PID for tie-breaking.

The result was an excessively high number of iterations in DeterminePartitionPlacement, which attempts to
assign partitions from rebalancing to reduce data communication.

On a milestone test problem on 256 cores, here's the effect:

Before patch:

    Level 1
    DeterminePartitionPlacement: Arbitration phase
    Number of arbitration rounds = 18
    Level 2
    DeterminePartitionPlacement: Arbitration phase
    Number of arbitration rounds = 9

After patch:

    Level 1
    DeterminePartitionPlacement: Arbitration phase
    Number of arbitration rounds = 6

    Level 2
    DeterminePartitionPlacement: Arbitration phase
    Number of arbitration rounds = 4
MueLu:  fixing shadow warning
MueLu: Changing seed for random tie-breaking.

This was run-dependent. This commit fixes the seed, based on each processor's PID.
So now results should be reproducible from run to run.

I've tested this on milestone problems on 256 and 2048 cores, and the results
are virtually the same, in terms of the number of arbitration rounds required during
the diffusive partition placement, as the previous seeding.

This required re-baselining one unit test.

Build/Test Cases Summary
Enabled Packages: MueLu
Disabled Packages: MOOCHO
Enabled all Forward Packages
0) MPI_DEBUG => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
1) SERIAL_RELEASE => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
2) MPI_SS_DEBUG => passed: passed=86,notpassed=0 (1.95 min)
Other local commits for this build/test group: 8562854, fc94ebe, 33d140a
Merge branch 'master' of software.sandia.gov:/space/git/Trilinos
Added a BARRIER_VIOLATION test to the gradient_test.
MueLu: fix for long int
MueLu: Disabling debugging output by default.

This shouldn't be on for optimized builds.

Build/Test Cases Summary
Enabled Packages: MueLu
Disabled Packages: MOOCHO
Enabled all Forward Packages
0) MPI_DEBUG => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
1) SERIAL_RELEASE => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
2) MPI_SS_DEBUG => passed: passed=86,notpassed=0 (9.86 min)
MueLu:  Removing some more debugging output.

Build/Test Cases Summary
Enabled Packages: MueLu
Disabled Packages: MOOCHO
Enabled all Forward Packages
0) MPI_DEBUG => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
1) SERIAL_RELEASE => Skipped configure, build, test due to no enabled packages! => Does not affect push readiness! (-1.00 min)
2) MPI_SS_DEBUG => passed: passed=86,notpassed=0 (9.84 min)