sajjad-moradi opened a new pull request #6976: URL: https://github.com/apache/incubator-pinot/pull/6976
## Description This PR is the fix for the issue described in #6966. Basically it reverts the changes recently made to deep store directory structure. ## Testing Done Deployed locally and verified that segments are created in tableName/segmentName directory structure instead of the current problematic directory structure tableName_OFFLINE/segmentName. Also segmentDownloadUrl in segment ZKMetadata points to the desired directory. Before: `"segment.offline.download.url": "http://172.18.164.3:9000/segments/baseballStats_OFFLINE/baseballStats_OFFLINE_0"` After: `"segment.offline.download.url": "http://172.18.164.3:9000/segments/baseballStats/baseballStats_OFFLINE_0"` The tests were done with an offline table and also a hybrid table to validated both realtime and offline segments. The query execution is also fine for both offline and realtime scenarios. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
