Uploaded image for project: 'Kudu'
  1. Kudu
  2. KUDU-1944

Breaking changes best suited for a Kudu major release (i.e. 2.0)

    XMLWordPrintableJSON

    Details

    • Type: New Feature
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 1.3.0
    • Fix Version/s: None
    • Component/s: api, client, master, tserver
    • Labels:
      None

      Description

      I thought it might be useful to start a list of all Kudu features or additions that we suspect may be breaking (i.e. may break client API/ABI compatibility, wire compatibility, on-disk compatibility, etc.) and should be reserved for a Kudu major release, where we could expect to tolerate such breakages.

      Here are a few candidates off the top of my head.

      Remove std:: classes from C++ client API

      The presence of std:: classes complicates maintenance of ABI compatibility w.r.t. libstdc++. Couple options here:

      • Replace std:: classes with simpler primitives (i.e. std::string -> void * + size_t).
      • Replace std:: classes with Kudu counterparts (i.e. std::string -> faststring).
      • Replace entire C++ API with a C-based API.
      • Add a C-based API but retain C++ API as header-only.

      I believe all of these break ABI compatibility and likely API compatibility too.

      Rework Java client async API

      When we first released Kudu 1.0 we thought about stabilizing the async Java API (i.e. all public classes that start with Async), but decided against it because we had no good use cases. If we were to do that now, we'd most certainly be breaking ABI/API compatibility for anyone using it in its current form.

      Rework status/error codes

      This one also comes up from time to time. It bemoans the coarse grainedness of Status' error codes and, depending on how its done, may break client ABI/API compatibility as well as wire compatibility.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                adar Adar Dembo
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated: