ovsdb-server: Remove unixctl transaction support.
authorBen Pfaff <blp@nicira.com>
Fri, 6 Nov 2009 21:36:17 +0000 (13:36 -0800)
committerBen Pfaff <blp@nicira.com>
Fri, 6 Nov 2009 23:33:47 +0000 (15:33 -0800)
Executing transactions over JSON-RPC makes more sense, now that it is
supported, so remove the older support for unixctl.

ovsdb/ovsdb-server.c

index 17a9970ea9e1d17ac33b27b546a44eb844a3b33c..908042d77804676ce406e9ba8deec363c11b8551 100644 (file)
@@ -48,8 +48,6 @@ static void parse_options(int argc, char *argv[], char **file_namep,
                           struct svec *active, struct svec *passive);
 static void usage(void) NO_RETURN;
 
-static void ovsdb_transact(struct unixctl_conn *, const char *args, void *db);
-
 int
 main(int argc, char *argv[])
 {
@@ -90,8 +88,6 @@ main(int argc, char *argv[])
         ovs_fatal(retval, "could not listen for control connections");
     }
 
-    unixctl_command_register("ovsdb/transact", ovsdb_transact, db);
-
     for (;;) {
         ovsdb_jsonrpc_server_run(jsonrpc);
         unixctl_server_run(unixctl);
@@ -198,26 +194,3 @@ usage(void)
     leak_checker_usage();
     exit(EXIT_SUCCESS);
 }
-\f
-static void
-ovsdb_transact(struct unixctl_conn *conn, const char *args, void *db_)
-{
-    struct ovsdb *db = db_;
-    struct json *request, *reply;
-    char *reply_string;
-
-    /* Parse JSON. */
-    request = json_from_string(args);
-    if (request->type == JSON_STRING) {
-        unixctl_command_reply(conn, 501, request->u.string);
-        json_destroy(request);
-        return;
-    }
-
-    /* Execute command. */
-    reply = ovsdb_execute(db, request, 0, NULL);
-    reply_string = json_to_string(reply, 0);
-    unixctl_command_reply(conn, 200, reply_string);
-    free(reply_string);
-    json_destroy(reply);
-}