MueLu: RepartitionFactory, FactoryManager: minor cleanup
MueLu: follow up on b33c945df6e8

Fixing similar issues in the Repartition Build test
MueLu: cleaning up Rebalancing

Now that Zoltan2 team improved the multijagged partitioner to work with
any number of subdomains, all the adjustments and calculations of the
number of subdomains and multijagged parameters are unnecessary.
MueLu: fixed Repartition unit tests

The factory for the user provided 'number of partitions' was incorrectly
set to Teuchos::null, instead of MueLu::NoFactory. As a result, the
number of partitions was generated anew.
add a --kill-server/--no-kill-server flag.  The flag defaults to kill which is the old behavior.
check_for_waiting_connection() - before inserting a new connection into the connection maps, delete any existing connection from the same peer.

Build/Test Cases Summary
Enabled Packages: Trios
0) MPI_DEBUG => Skipped configure, build, test due to no enabled packages! => 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 => passed: passed=24,notpassed=0 (3.01 min)
Other local commits for this build/test group: eb287af
Tpetra,Ifpack2: Fixed methods that were returning const RCP&.

It is a very bad idea for any function or method to return a smart
pointer by reference.  For example, if an instance method copies a
class' RCP member and returns the copy, the copy will fall out of scope.
If that method returns the copy by reference (const RCP& instead of
RCP), then it is returning something (the RCP) that doesn't necessarily
exist any more.  This can result in segfaults or other undefined
behavior; see e.g., commit a043e94d2149c19e076636c3604053ae25b4cbef.

Tpetra has been doing this for many years now, and Ifpack2 has followed
suit.  Only luck has saved us before.  Now that Tpetra is being used
more and more, it is becoming more important that we fix this once and
for all.  This commit _only_ fixes Tpetra and Ifpack2.  It WILL break
Xpetra, MueLu, Amesos2, and possibly other packages.  Thus, it is not
ready to push yet.  I will first have to fix those downstream packages
as best I can.  However, do remember that code that doesn't compile is
better than code that might segfault with no warning.
Tpetra: RowMatrixTransposer: Fixed ETI bug.

In a recent commit, I fixed RowMatrixTransposer explicit template
instantiation (ETI) for Node=ThrustGPUNode.  However, I botched the
copy-and-paste, and repeated LO=int,GO=int instead of making the
copied-and-pasted code have LO=int,GO=long.  This commit fixes that,
thus fixing errors for duplicated explicit instantiations.
MueLu: Fixed wrappers affected by my "const RCP&" fix in Tpetra.

In my last commit, I changed many Tpetra and Ifpack2 classes so that
instead of returning objects as "const RCP&" (which is unsafe), they
return "RCP".  This commit fixes MueLu classes that inherit from
Tpetra::Operator (one of the interfaces affected in my last commit).
It also catches some rarely used Tpetra::Operator subclasses that I
missed in my previous commits.

I also took the opportunity to fix a compiler warning in an otherwise
unaffected header file.

Build/Test Cases Summary
Enabled Packages: Tpetra, Xpetra, MueLu, Amesos2
Disabled Packages: MOOCHO,Rythmos,TrilinosCouplings,FEI,NOX
Enabled all Forward Packages
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 => FAILED: passed=412,notpassed=14 => Not ready to push! (10.25 min)
3) SERIAL_RELEASE_SS => FAILED: passed=394,notpassed=10 => Not ready to push! (15.78 min)
Other local commits for this build/test group: e7286c6, 100e534
WARNING: Forced the push!
Tpetra: CrsMatrix: Removed some repeated calls to getRowMap().

Since getRowMap() and similiar method now return RCP instead of "const
RCP&", it's a bit more important now to avoid unnecessary calls to
getRowMap() and similar methods.

Build/Test Cases Summary
Enabled Packages: Tpetra, Belos, Ifpack2, Xpetra, Amesos2
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=140,notpassed=0 (8.79 min)
3) SERIAL_RELEASE_SS => passed: passed=123,notpassed=0 (7.26 min)
Fixed bug in matrix-market reader when reading matrix in chunks.
Kokkos: Rename subpackage KokkosArray to KokkosCore

  Change file prefix for soure from 'KokkosArray_' to 'Kokkos_'
  Rename namespace KokkosArray to Kokkos
  Change KokkosArray_config.h to KokkosCore_config.h
Tpetra:  RowMatrixTransposer:  Add GO=unsignedint to ThrustGPU ETI.

This fixes linking errors coming from TrilinosCouplings (which is odd
because TrilinosCouplings doesn't use the ThrustGPUNode).
Tpetra: CrsMatrix: Make clone() work when params is null.

The clone() method currently asks for sublists of the input
RCP<ParameterList> params, without first checking if it is null.
This makes using the default second argument for params impossible.
This commit fixes that, so sublist(params,...) only gets called
if params is not null.

Build/Test Cases Summary
Enabled Packages: Tpetra, Belos, Ifpack2, Xpetra
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=135,notpassed=0 (5.42 min)
3) SERIAL_RELEASE_SS => passed: passed=118,notpassed=0 (5.79 min)
Fixed CMake expression enabling Clang compilers that was mis-typed
Merge branch 'master' of software.sandia.gov:/space/git/Trilinos
Stokhos:  Several fixes relating to KokkosArray renaming.

Fixes several configure and build errors, and should address continuous
testing build errors on the dashboard.
Update installation testing

Updated the list of packages that can be tested with the installation testing to
include all current packages in Trilinos. In addition all currently released
packages were added to the install build if they weren't already. Any packages
that aren't released are not in the install build as well as a few others that
have dependencies that are more difficult to meet or have no need of an install
like TrilinosFramework.
Move TPLs to SCOREC external repo.

Move TPLs for ACIS and Parasolid to SCOREC external
repo.

Build/Test Cases Summary
Enabled Packages: TrilinosFramework
Enabled all Forward Packages
0) MPI_DEBUG => passed: passed=109,notpassed=0 (2.46 min)
1) SERIAL_RELEASE => passed: passed=109,notpassed=0 (1.41 min)
2) MPI_RELEASE_SS => passed: passed=109,notpassed=0 (2.47 min)
Xpetra: added clone() functionality for Map and Matrix

In many cases, we want to construct some data on CPU and then migrate it
to GPU. The current way of doing that in Tpetra is to use clone()
functionality, which allows the change of Node type. This patch tries to
propagate that functionality into Xpetra, which is going to be exercised
by MueLu.