Uploaded image for project: 'Phoenix'
  1. Phoenix
  2. PHOENIX-3178

Row count incorrect for UPSERT SELECT when auto commit is false

Agile BoardAttach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 4.15.0, 5.1.0
    • None

    Description

      To reproduce, use the following test:

          @Test
          public void testRowCountWithNoAutoCommitOnUpsertSelect() throws Exception {
              Properties props = PropertiesUtil.deepCopy(TEST_PROPERTIES);
              props.setProperty(QueryServices.MUTATE_BATCH_SIZE_ATTRIB, Integer.toString(3));
              props.setProperty(QueryServices.SCAN_CACHE_SIZE_ATTRIB, Integer.toString(3));
              props.setProperty(QueryServices.SCAN_RESULT_CHUNK_SIZE, Integer.toString(3));
              Connection conn = DriverManager.getConnection(getUrl(), props);
              conn.setAutoCommit(false);
              conn.createStatement().execute("CREATE SEQUENCE keys");
              String tableName = generateRandomString();
              conn.createStatement().execute(
                  "CREATE TABLE " + tableName + " (pk INTEGER PRIMARY KEY, val INTEGER)");
      
              conn.createStatement().execute(
                  "UPSERT INTO " + tableName + " VALUES (NEXT VALUE FOR keys,1)");
              conn.commit();
              for (int i=0; i<6; i++) {
                  Statement stmt = conn.createStatement();
                  int upsertCount = stmt.executeUpdate(
                      "UPSERT INTO " + tableName + " SELECT NEXT VALUE FOR keys, val FROM " + tableName);
                  conn.commit();
                  assertEquals((int)Math.pow(2, i), upsertCount);
              }
              conn.close();
          }
      

      Attachments

        Issue Links

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            swaroopa Swaroopa Kadam
            jamestaylor James R. Taylor
            Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Issue deployment