Results 10 comments of guidao

Thank you for your comments. I would like to implement this feature, but have some confusion and would like to ask for your help. 1. The current pr implementation is...

DeepSource execution failed, but it was not introduced by this pr. Should I fix it?

Got it. I will only fix the problem introduced by this pr in order to avoid too many changes.

@Civil This pr is ready, can you help to review it?

> @guidao: Are you OK if I pick up your PR? If you have any updates - it would be great. Thanks! Please feel free to do so. We solved...

I encountered the same problem. ``` 2022-09-20T07:17:53.854Z INFO quickwit_metastore::metastore::grpc_metastore: 571: Adding gRPC address `ip1:7281` to `MetastoreGrpcClient`. 2022-09-20T07:18:03.854Z INFO quickwit_metastore::metastore::grpc_metastore: 571: Adding gRPC address `ip2:7281` to `MetastoreGrpcClient`. 2022-09-20T07:18:14.854Z INFO quickwit_metastore::metastore::grpc_metastore: 571:...

> FYI here: IMO this approach is very flexible and gives ability to override expand in every backend but IMO it's a bit overkill, I'll create separate PR similar to...

> PS: you probably mean [grafana/grafana#33694](https://github.com/grafana/grafana/pull/33694) ? I implemented having exact that use case in mind: `find?query=*.servers.*.cpu` will return `cpu`, `expand?query=*.servers.*.cpu` will return full names. Good, I also think the...

Hi @carlosroman, can you help review this pr? This is really a useful feature.

The example in #263 is a bit incorrect, in fact in our environment we are using a dns name, not a literal IP address. In #263 I was trying to...