david yuan

Results 39 comments of david yuan

> LGTM in general, can we add some tests? i'm not sure did this improvement could execute unit test? But i will try

@cxzl25 @yaooqinn what do you think about the unit test case in ha module(because we need start a zk docker to test the poll strategy)

hello, i'm interested with this issue, could you assigned to me? @FangYongs

@bowenliang123 @pan3793 @yaooqinn please check this issue, i cannot resolve it.

> Is hive catalog OK to use for this case? I think it should be ok,but the derby(memory or file) will has some compatible errors.

> Let's give it a try? org.apache.iceberg.hive.RuntimeMetaException: Failed to connect to Hive Metastore .... Caused by: java.lang.RuntimeException: Unable to instantiate org.apache.hadoop.hive.metastore.HiveMetaStoreClient .... Caused by: MetaException(message:Unable to open a test connection...

> This Derby means you try to initiate 2 db instances in the same directory, it can be fixed in many ways I think you are right,but i have tried...

> Iceberg has a pool for the Hive client, disabling it might help. sorry, i'm not good at derby, i cannot find the root case, there has a pr #7071...

> Hi team [@davidyuan1223](https://github.com/davidyuan1223) [@yaooqinn](https://github.com/yaooqinn) [@pan3793](https://github.com/pan3793), I hope you're all doing well. Any updates on this issue? Sorry. I'm busy recently. Maybe next month try to resolve it?