hive icon indicating copy to clipboard operation
hive copied to clipboard

HIVE-28658 Add Iceberg REST Catalog client support

Open zratkai opened this issue 11 months ago • 7 comments

Change-Id: I5bb2559f7ca602b71f8ca03c852e2deff1a1bc52

What changes were proposed in this pull request?

Iceberg REST client implementation added to support Iceberg REST server connection.

Why are the changes needed?

To support Iceberg REST server connection.

Does this PR introduce any user-facing change?

No.

Is the change a dependency upgrade?

How was this patch tested?

Unit test.

zratkai avatar Jan 31 '25 17:01 zratkai

@zratkai Thanks for your PR! Could you give an example of a test? Or the records & screenshots you've tested?

zhangbutao avatar Feb 05 '25 08:02 zhangbutao

Why are these methods don't do anything?

  @Override
  public void createCatalog(Catalog catalog)
      throws AlreadyExistsException, InvalidObjectException, MetaException, TException {
  }

  @Override
  public void alterCatalog(String catalogName, Catalog newCatalog)
      throws NoSuchObjectException, InvalidObjectException, MetaException, TException {

  }

  @Override
  public Catalog getCatalog(String catName) throws NoSuchObjectException, MetaException, TException {
    return null;
  }

  @Override
  public List<String> getCatalogs() throws MetaException, TException {
    return new LinkedList<>();
  }

  @Override
  public void dropCatalog(String catName)
      throws NoSuchObjectException, InvalidOperationException, MetaException, TException {

  }

  @Override
  public void dropCatalog(String catName, boolean ifExists) throws TException {

  }

difin avatar Apr 25 '25 14:04 difin

why so many sonar issues - 767 New issues?

deniskuzZ avatar May 28 '25 13:05 deniskuzZ

@zratkai , could you please address the blocker, high and medium level issues reported by sonar

deniskuzZ avatar Jun 12 '25 13:06 deniskuzZ

@zratkai, good luck with the PR. Frankly, I've had enough of the arrogance.

deniskuzZ avatar Jul 16 '25 08:07 deniskuzZ

If you continue work on this and reuse any part of my work, add my name as an author.

zratkai avatar Jul 21 '25 12:07 zratkai