mbox series

[ovs-dev,v6,0/1] ovsdb-idl: Address (partially) missing table and column issues.

Message ID 20210826225555.1739236-1-numans@ovn.org
Headers show
Series ovsdb-idl: Address (partially) missing table and column issues. | expand

Message

Numan Siddique Aug. 26, 2021, 10:55 p.m. UTC 
From: Numan Siddique <nusiddiq@redhat.com>

This patch partially addresses the transaction issues seen when
the C IDL client is running compiled with newer schema and ovsdb-server
is still running with older schema.

Patch 1: Adds the API to query for table and column names.


v5 -> v6
-----
  * Addresses the review comments from Ilya for v5 patch 1.
  * Patch 2 which addresses the transaction errors
    is dropped from v6 as this would require some more work.
    Patch 1 can be independently considered.

v4 -> v5
-----
  * Addressed the review comments from Ilya for v4 patch 1.
  * patch 2 is unchanged.

v3 -> v4
------
  * Addressed the review comment from Ilya for patch 2.
    ovsdb_idl_txn_insert() will not return NULL for missing tables.
 
v2 -> v3
------
  * Patch 2 is added in v3.
  * Patch 1 implementation in v2 was wrong.  This patch fixes it and
    enhances the test cases.

v1 -> v2
------
  * Added the test cases.


Numan Siddique (1):
  ovsdb-idl : Add APIs to query if a table and a column is present or
    not.

 lib/ovsdb-idl-provider.h |  4 ++
 lib/ovsdb-idl.c          | 49 +++++++++++++++++++++++
 lib/ovsdb-idl.h          |  5 ++-
 ovsdb/ovsdb-idlc.in      | 28 ++++++++++++-
 tests/idltest.ovsschema  |  9 +++++
 tests/idltest2.ovsschema |  7 ++++
 tests/ovsdb-idl.at       | 40 +++++++++++++++++++
 tests/test-ovsdb.c       | 86 ++++++++++++++++++++++++++++++++++++++++
 8 files changed, 226 insertions(+), 2 deletions(-)