use LOG_ABORT not just abort()

* abort() has the unfortunate effect that nothing is seen in the log,
  just an event (which is usually not displayed); so ops people don't
  see that the program is crashing at all.
* LOG_ABORT("message") will log an error with the message (and the file
  and line) before calling abort(), so it's easy to see what happened.
* add or move <vespa/log/log.h> include and LOG_SETUP lines before
  LOG_ABORT is used (or included).

1 files changed, 2 insertions, 2 deletions

diff --git a/vdslib/src/vespa/vdslib/distribution/distribution.cpp b/vdslib/src/vespa/vdslib/distribution/distribution.cpp
index 9b31d10d8e8..5a834929981 100644
--- a/vdslib/src/vespa/vdslib/distribution/distribution.cpp
+++ b/vdslib/src/vespa/vdslib/distribution/distribution.cpp
@@ -124,7 +124,7 @@ namespace {
             case ConfigDiskDistribution::MODULO_INDEX : return Distribution::MODULO_INDEX;
             case ConfigDiskDistribution::MODULO_KNUTH : return Distribution::MODULO_KNUTH;
         }
-        abort();
+        LOG_ABORT("should not be reached");
     }
     ConfigDiskDistribution toConfig(Distribution::DiskDistribution cfg) {
         switch (cfg) {
@@ -133,7 +133,7 @@ namespace {
             case Distribution::MODULO_INDEX : return ConfigDiskDistribution::MODULO_INDEX;
             case Distribution::MODULO_KNUTH : return ConfigDiskDistribution::MODULO_KNUTH;
         }
-        abort();
+        LOG_ABORT("should not be reached");
     }
 }