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

Row count incorrect for UPSERT SELECT when auto commit is false

    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

        1. PHOENIX-3178-4.x-HBase-1.4.patch
          4 kB
          Swaroopa Kadam
        2. PHOENIX-3178.patch
          4 kB
          Swaroopa Kadam

        Issue Links

          Activity

            People

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

              Dates

                Created:
                Updated:
                Resolved: