Xiang Fu
Xiang Fu
@ebyhr can you take another pass?
Thanks @martint !
> @Jackie-Jiang Can you please **re-run the CI**? because the `pinot-jdbc-client` (which is failing in CI ) is successful in local testing. Seems re-run doesn't pass
Can you add a sample data file with a decimal field and a test to ensure the file is correctly parsed?
> I've just made an analysis of the docker image. > > It seems that the three bigger layers are: > > * 344MBs of base image (jdk-slim), which is...
The PR is on the right track. One thing worth mentioning is that the get table name is the best effort. And if the tableName is null, the brokerSelector should...
We used to have the pipeline: https://github.com/apachepinot/pinot-fork/commit/2fe895373d8a9404a0d6bad2abef7b7e98e6d35f But the docker build for arm64 really takes too long on amd machines, all existing CI from GitHub action failed due to timeout.
Not yet. The original goal is to create a multi-arch docker image with the same tag. So building an arm image on amd machine takes about 4 hours. I think...
I've added a nightly build image for arm64: https://github.com/apachepinot/pinot-fork/actions/workflows/nightly-build-pinot-arm64-platform-docker-image.yml It takes about 5 hours to build. The next try is to make a multi-arch build.