Skip to content

Conversation

@moiseenkov
Copy link
Contributor

@moiseenkov moiseenkov commented May 2, 2023

Implemented MetastoreHivePartitionSensor that allows users to check the existence of Hive partitions in Dataproc Metastore via gRPC requests instead of thrift.

@moiseenkov moiseenkov force-pushed the hive_metastore_partition_grpc_sensor_main branch 2 times, most recently from c83574a to e759db7 Compare May 2, 2023 15:06
@moiseenkov moiseenkov force-pushed the hive_metastore_partition_grpc_sensor_main branch from e759db7 to f56a764 Compare May 24, 2023 10:03
@moiseenkov moiseenkov marked this pull request as ready for review May 24, 2023 10:03
@moiseenkov moiseenkov force-pushed the hive_metastore_partition_grpc_sensor_main branch 17 times, most recently from 6ca1877 to 264fb45 Compare May 31, 2023 07:13
@moiseenkov moiseenkov force-pushed the hive_metastore_partition_grpc_sensor_main branch 5 times, most recently from d11c396 to fae18d1 Compare June 2, 2023 08:02
@moiseenkov moiseenkov force-pushed the hive_metastore_partition_grpc_sensor_main branch from fae18d1 to 46ce383 Compare June 2, 2023 12:04
@moiseenkov
Copy link
Contributor Author

@potiuk , hi, could you help with code review please?

@moiseenkov moiseenkov force-pushed the hive_metastore_partition_grpc_sensor_main branch from 46ce383 to de58873 Compare June 2, 2023 14:32
@potiuk
Copy link
Member

potiuk commented Jun 4, 2023

@potiuk , hi, could you help with code review please?

I am quite busy recently, and I have no particular knowledge of that part, maybe someone else can help with it ? Let's see maybe someone will take a look soon.

@moiseenkov moiseenkov force-pushed the hive_metastore_partition_grpc_sensor_main branch 12 times, most recently from 2cfba11 to c2d7e60 Compare June 9, 2023 11:55
@potiuk potiuk force-pushed the hive_metastore_partition_grpc_sensor_main branch from c2d7e60 to 3e6d3e2 Compare June 11, 2023 18:16
@potiuk potiuk merged commit 810d467 into apache:main Jun 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants