frankxieke opened a new pull request #12309:
URL: https://github.com/apache/pulsar/pull/12309
### Motivation
* When using FileSystem Offloader, and you set the ledger retention policy,
after the offloaded ledger is outdated, however the data can not be deleted
from file system. This is because the datafile path is wrong.
* For example, in fact the datafile path is
"file:///Users/pulsar_nfs/public/default/persistent/test_pulsar_delta/449-775f0961-9719-4658-8357-6b0edbdef7a3",
but is mistaken formatted as
"file:///Users/pulsar_nfs/null/449-775f0961-9719-4658-8357-6b0edbdef7a3".
* The reason is when format the data path, "ManagedLedgerName" property in
"offloadDriverMetadata" is missing.
### Modifications
* Before format the data path, add "ManagedLedgerName" property in
"offloadDriverMetadata" map.
### Verifying this change
- [x] Make sure that the change passes the CI checks.
This change is already covered by existing tests.
### Does this pull request potentially affect one of the following parts:
- Dependencies (does it add or upgrade a dependency): (no)
- The public API: (no)
- The schema: (no)
- The default values of configurations: (no)
- The wire protocol: (no)
- The rest endpoints: (no)
- The admin cli options: (no)
- Anything that affects deployment: (no)
### Documentation
Need to update docs?
- [x] no-need-doc
fix bug, not affect user using the offload.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
