apache / apache/pinot

Implementing PinotClientTransport using Vert.x

Open
#10,924 5 comments 0 reactions 1 assignee Claimed by @lucifer4j View on GitHub
pinot-client
Dominant language
Java
Stars
6.1k
Forks
1.5k
Avg merge
2d 3h
Merged PRs (30d)
195

Description

Hi! I are trying to implement the PinotClientTransport interface as a part of creating a Vert.x Client adapter for the Pinot client. There are two changes we have identified so far that can help move in that direction.
1. https://github.com/apache/pinot/pull/10326
2. Making [BrokerResponse](https://github.com/apache/pinot/blob/master/pinot-clients/pinot-java-client/src/main/java/org/apache/pinot/client/BrokerResponse.java) a public class. Currently its package private but the public interface of PinotClientTransport returns it from the executeQuery and executeQueryAsync methods making it impossible to implement the interface externally.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.