Build #1,640

Build: #1640 was successful Child of CDAP-DRC-5622

Code commits

caskdata/cdap

  • wyzhang <wyzhang@google.com>

    wyzhang <wyzhang@google.com> cb093f26c6fa8e98a40b728b764ff190c1538e4a

    Merge pull request #13768 from cdapio/feature/CDAP-18627/wyzhang-leveldb-api-preview
    [CDAP-18627] Switch DefaultPreviewStore to use new LevelDBTableCore API

  • Wangyuan Zhang <wyzhang@google.com>

    Wangyuan Zhang <wyzhang@google.com> 291a794f9042943b43bc7c2dd992a07f1f955a34

    [CDAP-18627] Switch DefaultPreviewStore to use new LevelDBTableCore API              Also fix a test breakage for DefaultPreviewStoreTest
    Why:
    DefaultPreviewStore doesn't need to read/write/delete to levelDB at different versions.
    So switch to use new LevelDBTableCore API that operate at max version (i.e. always
    overwrite old value). This also provide some performance gain by getRow()->get()
    as getRow() internally perform iterator().next while get() is a point lookup
    for a specific version.

    • cdap-app-fabric/src/main/java/io/cdap/cdap/internal/app/store/preview/DefaultPreviewStore.java (version 291a794f9042943b43bc7c2dd992a07f1f955a34)
  • wyzhang <wyzhang@google.com>

    wyzhang <wyzhang@google.com> 168578a68c8edd2d90a617370c4c454520514dbf

    Merge pull request #13771 from cdapio/feature/CDAP-18627/wyzhang-leveldb-api-cleanup
    [CDAP-18627] Remove confusing createEndKey(byte[] row) which just calls createStartKey(row)

  • Wangyuan Zhang <wyzhang@google.com>

    Wangyuan Zhang <wyzhang@google.com> 9d448d63dbd14b6c886d86c2ed4001ab7568ffd1 m

    [CDAP-18627] Remove confusing createEndKey(byte[] row) which just call createStartKey(row)
    Why:
     private static byte[] createEndKey(byte[] row) {
        return createStartKey(row); // the first key of the stop is the first to be excluded
      }
    This method is confusing as it is NOT creating end key for a row,
    instead it assumes the row passed in is the end-row (i.e. excluded) for a scan.

    • cdap-data-fabric/src/main/java/io/cdap/cdap/data2/dataset2/lib/table/leveldb/LevelDBTableCore.java (version 9d448d63dbd14b6c886d86c2ed4001ab7568ffd1)