apache/pinot
Auf GitHub ansehenInconsistent segment file extension for TarGzPush and MetadataPush
Open
#10.375 geöffnet am 3. März 2023
good first issue
Repository-Metriken
- Stars
- (4.937 Stars)
- PR-Merge-Metriken
- (Durchschn. Merge 6T 7h) (186 gemergte PRs in 30 T)
Beschreibung
Our SegmentGenerationJobRunner adds tar.gz extension to the segment file name.
- TarGzPush: the controller side segment upload logic removes
tar.gzextension when copying data to deep storage. The segment file will be stored asfileName. - MetadataPush: we use pinotFS to copy file from local to dst. Segment file will be stored as
fileName.tar.gz.
It's going to be good if we can make the name convension the same. But, this will change the file name convension to the deepstorage. We should at least consider to add the config to remove the extension to have the clear migration path. (e.g. some people can start the cluster using TarGzPush and want to migrate to MetadataPush and keep the file names the same convention in the deep storage)