ovsdb: Make ovsdb_column_set_from_json() take table schema instead of table.
[openvswitch] / ovsdb / execution.c
1 /* Copyright (c) 2009, 2010, 2011 Nicira Networks
2  *
3  * Licensed under the Apache License, Version 2.0 (the "License");
4  * you may not use this file except in compliance with the License.
5  * You may obtain a copy of the License at:
6  *
7  *     http://www.apache.org/licenses/LICENSE-2.0
8  *
9  * Unless required by applicable law or agreed to in writing, software
10  * distributed under the License is distributed on an "AS IS" BASIS,
11  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
12  * See the License for the specific language governing permissions and
13  * limitations under the License.
14  */
15
16 #include <config.h>
17
18 #include <assert.h>
19 #include <limits.h>
20
21 #include "column.h"
22 #include "condition.h"
23 #include "file.h"
24 #include "json.h"
25 #include "mutation.h"
26 #include "ovsdb-data.h"
27 #include "ovsdb-error.h"
28 #include "ovsdb-parser.h"
29 #include "ovsdb.h"
30 #include "query.h"
31 #include "row.h"
32 #include "table.h"
33 #include "timeval.h"
34 #include "transaction.h"
35
36 struct ovsdb_execution {
37     struct ovsdb *db;
38     struct ovsdb_txn *txn;
39     struct ovsdb_symbol_table *symtab;
40     bool durable;
41
42     /* Triggers. */
43     long long int elapsed_msec;
44     long long int timeout_msec;
45 };
46
47 typedef struct ovsdb_error *ovsdb_operation_executor(struct ovsdb_execution *,
48                                                      struct ovsdb_parser *,
49                                                      struct json *result);
50
51 static ovsdb_operation_executor ovsdb_execute_insert;
52 static ovsdb_operation_executor ovsdb_execute_select;
53 static ovsdb_operation_executor ovsdb_execute_update;
54 static ovsdb_operation_executor ovsdb_execute_mutate;
55 static ovsdb_operation_executor ovsdb_execute_delete;
56 static ovsdb_operation_executor ovsdb_execute_wait;
57 static ovsdb_operation_executor ovsdb_execute_commit;
58 static ovsdb_operation_executor ovsdb_execute_abort;
59 static ovsdb_operation_executor ovsdb_execute_comment;
60
61 static ovsdb_operation_executor *
62 lookup_executor(const char *name)
63 {
64     struct ovsdb_operation {
65         const char *name;
66         ovsdb_operation_executor *executor;
67     };
68
69     static const struct ovsdb_operation operations[] = {
70         { "insert", ovsdb_execute_insert },
71         { "select", ovsdb_execute_select },
72         { "update", ovsdb_execute_update },
73         { "mutate", ovsdb_execute_mutate },
74         { "delete", ovsdb_execute_delete },
75         { "wait", ovsdb_execute_wait },
76         { "commit", ovsdb_execute_commit },
77         { "abort", ovsdb_execute_abort },
78         { "comment", ovsdb_execute_comment },
79     };
80
81     size_t i;
82
83     for (i = 0; i < ARRAY_SIZE(operations); i++) {
84         const struct ovsdb_operation *c = &operations[i];
85         if (!strcmp(c->name, name)) {
86             return c->executor;
87         }
88     }
89     return NULL;
90 }
91
92 struct json *
93 ovsdb_execute(struct ovsdb *db, const struct json *params,
94               long long int elapsed_msec, long long int *timeout_msec)
95 {
96     struct ovsdb_execution x;
97     struct ovsdb_error *error;
98     struct json *results;
99     size_t n_operations;
100     size_t i;
101
102     if (params->type != JSON_ARRAY
103         || !params->u.array.n
104         || params->u.array.elems[0]->type != JSON_STRING
105         || strcmp(params->u.array.elems[0]->u.string, db->schema->name)) {
106         if (params->type != JSON_ARRAY) {
107             error = ovsdb_syntax_error(params, NULL, "array expected");
108         } else {
109             error = ovsdb_syntax_error(params, NULL, "database name expected "
110                                        "as first parameter");
111         }
112
113         results = ovsdb_error_to_json(error);
114         ovsdb_error_destroy(error);
115         return results;
116     }
117
118     x.db = db;
119     x.txn = ovsdb_txn_create(db);
120     x.symtab = ovsdb_symbol_table_create();
121     x.durable = false;
122     x.elapsed_msec = elapsed_msec;
123     x.timeout_msec = LLONG_MAX;
124     results = NULL;
125
126     results = json_array_create_empty();
127     n_operations = params->u.array.n - 1;
128     error = NULL;
129     for (i = 1; i <= n_operations; i++) {
130         struct json *operation = params->u.array.elems[i];
131         struct ovsdb_error *parse_error;
132         struct ovsdb_parser parser;
133         struct json *result;
134         const struct json *op;
135
136         /* Parse and execute operation. */
137         ovsdb_parser_init(&parser, operation,
138                           "ovsdb operation %zu of %zu", i, n_operations);
139         op = ovsdb_parser_member(&parser, "op", OP_ID);
140         result = json_object_create();
141         if (op) {
142             const char *op_name = json_string(op);
143             ovsdb_operation_executor *executor = lookup_executor(op_name);
144             if (executor) {
145                 error = executor(&x, &parser, result);
146             } else {
147                 ovsdb_parser_raise_error(&parser, "No operation \"%s\"",
148                                          op_name);
149             }
150         } else {
151             assert(ovsdb_parser_has_error(&parser));
152         }
153
154         /* A parse error overrides any other error.
155          * An error overrides any other result. */
156         parse_error = ovsdb_parser_finish(&parser);
157         if (parse_error) {
158             ovsdb_error_destroy(error);
159             error = parse_error;
160         }
161         if (error) {
162             json_destroy(result);
163             result = ovsdb_error_to_json(error);
164         }
165         if (error && !strcmp(ovsdb_error_get_tag(error), "not supported")
166             && timeout_msec) {
167             ovsdb_txn_abort(x.txn);
168             *timeout_msec = x.timeout_msec;
169
170             json_destroy(result);
171             json_destroy(results);
172             results = NULL;
173             goto exit;
174         }
175
176         /* Add result to array. */
177         json_array_add(results, result);
178         if (error) {
179             break;
180         }
181     }
182
183     if (!error) {
184         error = ovsdb_txn_commit(x.txn, x.durable);
185         if (error) {
186             json_array_add(results, ovsdb_error_to_json(error));
187         }
188     } else {
189         ovsdb_txn_abort(x.txn);
190     }
191
192     while (json_array(results)->n < n_operations) {
193         json_array_add(results, json_null_create());
194     }
195
196 exit:
197     ovsdb_error_destroy(error);
198     ovsdb_symbol_table_destroy(x.symtab);
199
200     return results;
201 }
202
203 static struct ovsdb_error *
204 ovsdb_execute_commit(struct ovsdb_execution *x, struct ovsdb_parser *parser,
205                      struct json *result OVS_UNUSED)
206 {
207     const struct json *durable;
208
209     durable = ovsdb_parser_member(parser, "durable", OP_BOOLEAN);
210     if (durable && json_boolean(durable)) {
211         x->durable = true;
212     }
213     return NULL;
214 }
215
216 static struct ovsdb_error *
217 ovsdb_execute_abort(struct ovsdb_execution *x OVS_UNUSED,
218                     struct ovsdb_parser *parser OVS_UNUSED,
219                     struct json *result OVS_UNUSED)
220 {
221     return ovsdb_error("aborted", "aborted by request");
222 }
223
224 static struct ovsdb_table *
225 parse_table(struct ovsdb_execution *x,
226             struct ovsdb_parser *parser, const char *member)
227 {
228     struct ovsdb_table *table;
229     const char *table_name;
230     const struct json *json;
231
232     json = ovsdb_parser_member(parser, member, OP_ID);
233     if (!json) {
234         return NULL;
235     }
236     table_name = json_string(json);
237
238     table = shash_find_data(&x->db->tables, table_name);
239     if (!table) {
240         ovsdb_parser_raise_error(parser, "No table named %s.", table_name);
241     }
242     return table;
243 }
244
245 static WARN_UNUSED_RESULT struct ovsdb_error *
246 parse_row(const struct json *json, const struct ovsdb_table *table,
247           struct ovsdb_symbol_table *symtab,
248           struct ovsdb_row **rowp, struct ovsdb_column_set *columns)
249 {
250     struct ovsdb_error *error;
251     struct ovsdb_row *row;
252
253     *rowp = NULL;
254
255     if (!table) {
256         return OVSDB_BUG("null table");
257     }
258     if (!json) {
259         return OVSDB_BUG("null row");
260     }
261
262     row = ovsdb_row_create(table);
263     error = ovsdb_row_from_json(row, json, symtab, columns);
264     if (error) {
265         ovsdb_row_destroy(row);
266         return error;
267     } else {
268         *rowp = row;
269         return NULL;
270     }
271 }
272
273 static struct ovsdb_error *
274 ovsdb_execute_insert(struct ovsdb_execution *x, struct ovsdb_parser *parser,
275                      struct json *result)
276 {
277     struct ovsdb_table *table;
278     struct ovsdb_row *row = NULL;
279     const struct json *uuid_name, *row_json;
280     struct ovsdb_error *error;
281     struct uuid row_uuid;
282
283     table = parse_table(x, parser, "table");
284     uuid_name = ovsdb_parser_member(parser, "uuid-name", OP_ID | OP_OPTIONAL);
285     row_json = ovsdb_parser_member(parser, "row", OP_OBJECT);
286     error = ovsdb_parser_get_error(parser);
287     if (error) {
288         return error;
289     }
290
291     if (uuid_name) {
292         struct ovsdb_symbol *symbol;
293
294         symbol = ovsdb_symbol_table_insert(x->symtab, json_string(uuid_name));
295         if (symbol->created) {
296             return ovsdb_syntax_error(uuid_name, "duplicate uuid-name",
297                                       "This \"uuid-name\" appeared on an "
298                                       "earlier \"insert\" operation.");
299         }
300         row_uuid = symbol->uuid;
301         symbol->created = true;
302     } else {
303         uuid_generate(&row_uuid);
304     }
305
306     if (!error) {
307         error = parse_row(row_json, table, x->symtab, &row, NULL);
308     }
309     if (!error) {
310         /* Check constraints for columns not included in "row", in case the
311          * default values do not satisfy the constraints.  We could check only
312          * the columns that have their default values by supplying an
313          * ovsdb_column_set to parse_row() above, but I suspect that this is
314          * cheaper.  */
315         const struct shash_node *node;
316
317         SHASH_FOR_EACH (node, &table->schema->columns) {
318             const struct ovsdb_column *column = node->data;
319             const struct ovsdb_datum *datum = &row->fields[column->index];
320
321             /* If there are 0 keys or pairs, there's nothing to check.
322              * If there is 1, it might be a default value.
323              * If there are more, it can't be a default value, so the value has
324              * already been checked. */
325             if (datum->n == 1) {
326                 error = ovsdb_datum_check_constraints(datum, &column->type);
327                 if (error) {
328                     ovsdb_row_destroy(row);
329                     break;
330                 }
331             }
332         }
333     }
334     if (!error) {
335         *ovsdb_row_get_uuid_rw(row) = row_uuid;
336         ovsdb_txn_row_insert(x->txn, row);
337         json_object_put(result, "uuid",
338                         ovsdb_datum_to_json(&row->fields[OVSDB_COL_UUID],
339                                             &ovsdb_type_uuid));
340     }
341     return error;
342 }
343
344 static struct ovsdb_error *
345 ovsdb_execute_select(struct ovsdb_execution *x, struct ovsdb_parser *parser,
346                      struct json *result)
347 {
348     struct ovsdb_table *table;
349     const struct json *where, *columns_json, *sort_json;
350     struct ovsdb_condition condition = OVSDB_CONDITION_INITIALIZER;
351     struct ovsdb_column_set columns = OVSDB_COLUMN_SET_INITIALIZER;
352     struct ovsdb_column_set sort = OVSDB_COLUMN_SET_INITIALIZER;
353     struct ovsdb_error *error;
354
355     table = parse_table(x, parser, "table");
356     where = ovsdb_parser_member(parser, "where", OP_ARRAY);
357     columns_json = ovsdb_parser_member(parser, "columns",
358                                        OP_ARRAY | OP_OPTIONAL);
359     sort_json = ovsdb_parser_member(parser, "sort", OP_ARRAY | OP_OPTIONAL);
360
361     error = ovsdb_parser_get_error(parser);
362     if (!error) {
363         error = ovsdb_condition_from_json(table->schema, where, x->symtab,
364                                           &condition);
365     }
366     if (!error) {
367         error = ovsdb_column_set_from_json(columns_json, table->schema,
368                                            &columns);
369     }
370     if (!error) {
371         error = ovsdb_column_set_from_json(sort_json, table->schema, &sort);
372     }
373     if (!error) {
374         struct ovsdb_row_set rows = OVSDB_ROW_SET_INITIALIZER;
375
376         ovsdb_query_distinct(table, &condition, &columns, &rows);
377         ovsdb_row_set_sort(&rows, &sort);
378         json_object_put(result, "rows",
379                         ovsdb_row_set_to_json(&rows, &columns));
380
381         ovsdb_row_set_destroy(&rows);
382     }
383
384     ovsdb_column_set_destroy(&columns);
385     ovsdb_column_set_destroy(&sort);
386     ovsdb_condition_destroy(&condition);
387
388     return error;
389 }
390
391 struct update_row_cbdata {
392     size_t n_matches;
393     struct ovsdb_txn *txn;
394     const struct ovsdb_row *row;
395     const struct ovsdb_column_set *columns;
396 };
397
398 static bool
399 update_row_cb(const struct ovsdb_row *row, void *ur_)
400 {
401     struct update_row_cbdata *ur = ur_;
402
403     ur->n_matches++;
404     if (!ovsdb_row_equal_columns(row, ur->row, ur->columns)) {
405         ovsdb_row_update_columns(ovsdb_txn_row_modify(ur->txn, row),
406                                  ur->row, ur->columns);
407     }
408
409     return true;
410 }
411
412 static struct ovsdb_error *
413 ovsdb_execute_update(struct ovsdb_execution *x, struct ovsdb_parser *parser,
414                      struct json *result)
415 {
416     struct ovsdb_table *table;
417     const struct json *where, *row_json;
418     struct ovsdb_condition condition = OVSDB_CONDITION_INITIALIZER;
419     struct ovsdb_column_set columns = OVSDB_COLUMN_SET_INITIALIZER;
420     struct ovsdb_row *row = NULL;
421     struct update_row_cbdata ur;
422     struct ovsdb_error *error;
423
424     table = parse_table(x, parser, "table");
425     where = ovsdb_parser_member(parser, "where", OP_ARRAY);
426     row_json = ovsdb_parser_member(parser, "row", OP_OBJECT);
427     error = ovsdb_parser_get_error(parser);
428     if (!error) {
429         error = parse_row(row_json, table, x->symtab, &row, &columns);
430     }
431     if (!error) {
432         error = ovsdb_condition_from_json(table->schema, where, x->symtab,
433                                           &condition);
434     }
435     if (!error) {
436         ur.n_matches = 0;
437         ur.txn = x->txn;
438         ur.row = row;
439         ur.columns = &columns;
440         ovsdb_query(table, &condition, update_row_cb, &ur);
441         json_object_put(result, "count", json_integer_create(ur.n_matches));
442     }
443
444     ovsdb_row_destroy(row);
445     ovsdb_column_set_destroy(&columns);
446     ovsdb_condition_destroy(&condition);
447
448     return error;
449 }
450
451 struct mutate_row_cbdata {
452     size_t n_matches;
453     struct ovsdb_txn *txn;
454     const struct ovsdb_mutation_set *mutations;
455     struct ovsdb_error **error;
456 };
457
458 static bool
459 mutate_row_cb(const struct ovsdb_row *row, void *mr_)
460 {
461     struct mutate_row_cbdata *mr = mr_;
462
463     mr->n_matches++;
464     *mr->error = ovsdb_mutation_set_execute(ovsdb_txn_row_modify(mr->txn, row),
465                                             mr->mutations);
466     return *mr->error == NULL;
467 }
468
469 static struct ovsdb_error *
470 ovsdb_execute_mutate(struct ovsdb_execution *x, struct ovsdb_parser *parser,
471                      struct json *result)
472 {
473     struct ovsdb_table *table;
474     const struct json *where;
475     const struct json *mutations_json;
476     struct ovsdb_condition condition = OVSDB_CONDITION_INITIALIZER;
477     struct ovsdb_mutation_set mutations = OVSDB_MUTATION_SET_INITIALIZER;
478     struct ovsdb_row *row = NULL;
479     struct mutate_row_cbdata mr;
480     struct ovsdb_error *error;
481
482     table = parse_table(x, parser, "table");
483     where = ovsdb_parser_member(parser, "where", OP_ARRAY);
484     mutations_json = ovsdb_parser_member(parser, "mutations", OP_ARRAY);
485     error = ovsdb_parser_get_error(parser);
486     if (!error) {
487         error = ovsdb_mutation_set_from_json(table->schema, mutations_json,
488                                              x->symtab, &mutations);
489     }
490     if (!error) {
491         error = ovsdb_condition_from_json(table->schema, where, x->symtab,
492                                           &condition);
493     }
494     if (!error) {
495         mr.n_matches = 0;
496         mr.txn = x->txn;
497         mr.mutations = &mutations;
498         mr.error = &error;
499         ovsdb_query(table, &condition, mutate_row_cb, &mr);
500         json_object_put(result, "count", json_integer_create(mr.n_matches));
501     }
502
503     ovsdb_row_destroy(row);
504     ovsdb_mutation_set_destroy(&mutations);
505     ovsdb_condition_destroy(&condition);
506
507     return error;
508 }
509
510 struct delete_row_cbdata {
511     size_t n_matches;
512     const struct ovsdb_table *table;
513     struct ovsdb_txn *txn;
514 };
515
516 static bool
517 delete_row_cb(const struct ovsdb_row *row, void *dr_)
518 {
519     struct delete_row_cbdata *dr = dr_;
520
521     dr->n_matches++;
522     ovsdb_txn_row_delete(dr->txn, row);
523
524     return true;
525 }
526
527 static struct ovsdb_error *
528 ovsdb_execute_delete(struct ovsdb_execution *x, struct ovsdb_parser *parser,
529                      struct json *result)
530 {
531     struct ovsdb_table *table;
532     const struct json *where;
533     struct ovsdb_condition condition = OVSDB_CONDITION_INITIALIZER;
534     struct ovsdb_error *error;
535
536     where = ovsdb_parser_member(parser, "where", OP_ARRAY);
537     table = parse_table(x, parser, "table");
538     error = ovsdb_parser_get_error(parser);
539     if (!error) {
540         error = ovsdb_condition_from_json(table->schema, where, x->symtab,
541                                           &condition);
542     }
543     if (!error) {
544         struct delete_row_cbdata dr;
545
546         dr.n_matches = 0;
547         dr.table = table;
548         dr.txn = x->txn;
549         ovsdb_query(table, &condition, delete_row_cb, &dr);
550
551         json_object_put(result, "count", json_integer_create(dr.n_matches));
552     }
553
554     ovsdb_condition_destroy(&condition);
555
556     return error;
557 }
558
559 struct wait_auxdata {
560     struct ovsdb_row_hash *actual;
561     struct ovsdb_row_hash *expected;
562     bool *equal;
563 };
564
565 static bool
566 ovsdb_execute_wait_query_cb(const struct ovsdb_row *row, void *aux_)
567 {
568     struct wait_auxdata *aux = aux_;
569
570     if (ovsdb_row_hash_contains(aux->expected, row)) {
571         ovsdb_row_hash_insert(aux->actual, row);
572         return true;
573     } else {
574         /* The query row isn't in the expected result set, so the actual and
575          * expected results sets definitely differ and we can short-circuit the
576          * rest of the query. */
577         *aux->equal = false;
578         return false;
579     }
580 }
581
582 static struct ovsdb_error *
583 ovsdb_execute_wait(struct ovsdb_execution *x, struct ovsdb_parser *parser,
584                    struct json *result OVS_UNUSED)
585 {
586     struct ovsdb_table *table;
587     const struct json *timeout, *where, *columns_json, *until, *rows;
588     struct ovsdb_condition condition = OVSDB_CONDITION_INITIALIZER;
589     struct ovsdb_column_set columns = OVSDB_COLUMN_SET_INITIALIZER;
590     struct ovsdb_row_hash expected = OVSDB_ROW_HASH_INITIALIZER(expected);
591     struct ovsdb_row_hash actual = OVSDB_ROW_HASH_INITIALIZER(actual);
592     struct ovsdb_error *error;
593     struct wait_auxdata aux;
594     long long int timeout_msec = 0;
595     size_t i;
596
597     timeout = ovsdb_parser_member(parser, "timeout", OP_NUMBER | OP_OPTIONAL);
598     where = ovsdb_parser_member(parser, "where", OP_ARRAY);
599     columns_json = ovsdb_parser_member(parser, "columns",
600                                        OP_ARRAY | OP_OPTIONAL);
601     until = ovsdb_parser_member(parser, "until", OP_STRING);
602     rows = ovsdb_parser_member(parser, "rows", OP_ARRAY);
603     table = parse_table(x, parser, "table");
604     error = ovsdb_parser_get_error(parser);
605     if (!error) {
606         error = ovsdb_condition_from_json(table->schema, where, x->symtab,
607                                           &condition);
608     }
609     if (!error) {
610         error = ovsdb_column_set_from_json(columns_json, table->schema,
611                                            &columns);
612     }
613     if (!error) {
614         if (timeout) {
615             timeout_msec = MIN(LLONG_MAX, json_real(timeout));
616             if (timeout_msec < 0) {
617                 error = ovsdb_syntax_error(timeout, NULL,
618                                            "timeout must be nonnegative");
619             } else if (timeout_msec < x->timeout_msec) {
620                 x->timeout_msec = timeout_msec;
621             }
622         } else {
623             timeout_msec = LLONG_MAX;
624         }
625         if (strcmp(json_string(until), "==")
626             && strcmp(json_string(until), "!=")) {
627             error = ovsdb_syntax_error(until, NULL,
628                                        "\"until\" must be \"==\" or \"!=\"");
629         }
630     }
631     if (!error) {
632         /* Parse "rows" into 'expected'. */
633         ovsdb_row_hash_init(&expected, &columns);
634         for (i = 0; i < rows->u.array.n; i++) {
635             struct ovsdb_row *row;
636
637             row = ovsdb_row_create(table);
638             error = ovsdb_row_from_json(row, rows->u.array.elems[i], x->symtab,
639                                         NULL);
640             if (error) {
641                 break;
642             }
643
644             if (!ovsdb_row_hash_insert(&expected, row)) {
645                 /* XXX Perhaps we should abort with an error or log a
646                  * warning. */
647                 ovsdb_row_destroy(row);
648             }
649         }
650     }
651     if (!error) {
652         /* Execute query. */
653         bool equal = true;
654         ovsdb_row_hash_init(&actual, &columns);
655         aux.actual = &actual;
656         aux.expected = &expected;
657         aux.equal = &equal;
658         ovsdb_query(table, &condition, ovsdb_execute_wait_query_cb, &aux);
659         if (equal) {
660             /* We know that every row in 'actual' is also in 'expected'.  We
661              * also know that all of the rows in 'actual' are distinct and that
662              * all of the rows in 'expected' are distinct.  Therefore, if
663              * 'actual' and 'expected' have the same number of rows, then they
664              * have the same content. */
665             size_t n_actual = ovsdb_row_hash_count(&actual);
666             size_t n_expected = ovsdb_row_hash_count(&expected);
667             equal = n_actual == n_expected;
668         }
669         if (!strcmp(json_string(until), "==") != equal) {
670             if (timeout && x->elapsed_msec >= timeout_msec) {
671                 if (x->elapsed_msec) {
672                     error = ovsdb_error("timed out",
673                                         "\"wait\" timed out after %lld ms",
674                                         x->elapsed_msec);
675                 } else {
676                     error = ovsdb_error("timed out", "\"wait\" timed out");
677                 }
678             } else {
679                 /* ovsdb_execute() will change this, if triggers really are
680                  * supported. */
681                 error = ovsdb_error("not supported", "triggers not supported");
682             }
683         }
684     }
685
686
687     ovsdb_row_hash_destroy(&expected, true);
688     ovsdb_row_hash_destroy(&actual, false);
689     ovsdb_column_set_destroy(&columns);
690     ovsdb_condition_destroy(&condition);
691
692     return error;
693 }
694
695 static struct ovsdb_error *
696 ovsdb_execute_comment(struct ovsdb_execution *x, struct ovsdb_parser *parser,
697                       struct json *result OVS_UNUSED)
698 {
699     const struct json *comment;
700
701     comment = ovsdb_parser_member(parser, "comment", OP_STRING);
702     if (!comment) {
703         return NULL;
704     }
705     ovsdb_txn_add_comment(x->txn, json_string(comment));
706
707     return NULL;
708 }