Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-3713

CRUD operations against Kudu tables should report the number of modified rows

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: Impala 2.6.0
    • Fix Version/s: Impala 2.8.0
    • Component/s: Backend
    • Labels:

      Description

      Currently, only INSERT DML operations report the number of modified rows at all. (The reporting is limited to the shell and the profile because HS2 has no way to return this information, see IMPALA-1789 which is not in scope for this JIRA).

      Other DML (DELETE, UPDATE, UPSERT) do not report the number of modified rows at all, but they should at least report the number of modified rows in the same way that INSERT does.

        Issue Links

          Activity

          Hide
          mjacobs Matthew Jacobs added a comment -

          commit 77a2941a42b3023d22f3cc7b2db01a94b4ddec2f
          Author: Matthew Jacobs <mj@cloudera.com>
          Date: Tue Nov 15 19:21:32 2016 -0800

          IMPALA-3713,IMPALA-4439: Fix Kudu DML shell reporting

          Adds support in the shell to report the number of modified
          rows for all DML operations, as well as the number of rows
          with errors.

          Testing: Added shell tests.

          Change-Id: I3d3d7aa8d176e03ea58fb00f2a81fb3e34965aa1
          Reviewed-on: http://gerrit.cloudera.org:8080/5103
          Reviewed-by: Alex Behm <alex.behm@cloudera.com>
          Tested-by: Internal Jenkins

          Show
          mjacobs Matthew Jacobs added a comment - commit 77a2941a42b3023d22f3cc7b2db01a94b4ddec2f Author: Matthew Jacobs <mj@cloudera.com> Date: Tue Nov 15 19:21:32 2016 -0800 IMPALA-3713 , IMPALA-4439 : Fix Kudu DML shell reporting Adds support in the shell to report the number of modified rows for all DML operations, as well as the number of rows with errors. Testing: Added shell tests. Change-Id: I3d3d7aa8d176e03ea58fb00f2a81fb3e34965aa1 Reviewed-on: http://gerrit.cloudera.org:8080/5103 Reviewed-by: Alex Behm <alex.behm@cloudera.com> Tested-by: Internal Jenkins

            People

            • Assignee:
              mjacobs Matthew Jacobs
              Reporter:
              dtsirogiannis Dimitris Tsirogiannis
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development