Remove UT libraries and clean up remaining UT datastructures (#1230)

* Remove UT string include from redis

* Remove UT string include from DB tests

* Modify TaskSpec_print to remove UT string

* Remove UT libraries

src/common/state/redis.cc

@@ -11,8 +11,6 @@ extern "C" {
 #include "hiredis/adapters/ae.h"
 }
 
-#include "utstring.h"
-
 #include "common.h"
 #include "db.h"
 #include "db_client_table.h"

src/common/task.h

@@ -4,7 +4,8 @@
 #include <stddef.h>
 #include <stdint.h>
 #include "common.h"
-#include "utstring.h"
+
+#include <string>
 
 #include "format/common_generated.h"
 
@@ -360,10 +361,9 @@ ObjectID task_compute_put_id(TaskID task_id, int64_t put_index);
  * Print the task as a humanly readable string.
  *
  * @param spec The task_spec in question.
- * @param output The buffer to write the string to.
- * @return Void.
+ * @return The humanly readable string.
  */
-void TaskSpec_print(TaskSpec *spec, UT_string *output);
+std::string TaskSpec_print(TaskSpec *spec);
 
 /**
  * Create a copy of the task spec. Must be freed with TaskSpec_free after use.

src/common/test/db_tests.cc

@@ -15,8 +15,6 @@
 #include "state/redis.h"
 #include "task.h"
 
-#include "utstring.h"
-
 SUITE(db_tests);
 
 TaskBuilder *g_task_builder = NULL;