Skip to content

[fix][admin] Fix can't delete tenant for v1#22550

Merged
merlimat merged 1 commit into
apache:masterfrom
Technoboy-:fix-delete-v1-tenant-bug
Apr 23, 2024
Merged

[fix][admin] Fix can't delete tenant for v1#22550
merlimat merged 1 commit into
apache:masterfrom
Technoboy-:fix-delete-v1-tenant-bug

Conversation

@Technoboy-

Copy link
Copy Markdown
Contributor

Motivation

Can't delete tenant for v1. Find this bug when fixing #22547.

Caused by: org.apache.pulsar.metadata.api.MetadataStoreException$ContentDeserializationException: Failed to deserialize payload for key '/admin/policies/p1/c1'
	at org.apache.pulsar.metadata.cache.impl.MetadataCacheImpl.lambda$readValueFromStore$0(MetadataCacheImpl.java:115)
	at java.base/java.util.concurrent.CompletableFuture$UniCompose.tryFire(CompletableFuture.java:1150)
	... 12 more
Caused by: com.fasterxml.jackson.databind.exc.MismatchedInputException: No content to map due to end-of-input
 at [Source: (byte[])""; line: 1, column: 0]
	at com.fasterxml.jackson.databind.exc.MismatchedInputException.from(MismatchedInputException.java:59)
	at com.fasterxml.jackson.databind.DeserializationContext.reportInputMismatch(DeserializationContext.java:1746)
	at com.fasterxml.jackson.databind.ObjectReader._initForReading(ObjectReader.java:360)
	at com.fasterxml.jackson.databind.ObjectReader._bindAndClose(ObjectReader.java:2095)
	at com.fasterxml.jackson.databind.ObjectReader.readValue(ObjectReader.java:1583)
	at org.apache.pulsar.metadata.cache.impl.JSONMetadataSerdeSimpleType.deserialize(JSONMetadataSerdeSimpleType.java:46)
	at org.apache.pulsar.metadata.cache.impl.MetadataCacheImpl.lambda$readValueFromStore$0(MetadataCacheImpl.java:111)
	... 13 more

2024-04-22T20:24:34,580 - INFO  - [configuration-metadata-store-4-1:ResourceGroupNamespaceConfigListener] - Metadata store notification: Path /admin/policies/p1/c1, Type Deleted
2024-04-22T20:24:34,594 - ERROR - [metadata-store-2-1:TenantsBase] - [pass.pass] Failed to delete tenant p1
org.apache.pulsar.metadata.api.MetadataStoreException: org.apache.zookeeper.KeeperException$NotEmptyException: KeeperErrorCode = Directory not empty for /managed-ledgers/p1
	at org.apache.pulsar.metadata.impl.ZKMetadataStore.getException(ZKMetadataStore.java:476) ~[classes/:?]
	at org.apache.pulsar.metadata.impl.ZKMetadataStore.handleDeleteResult(ZKMetadataStore.java:304) ~[classes/:?]
	at org.apache.pulsar.metadata.impl.ZKMetadataStore.lambda$batchOperation$5(ZKMetadataStore.java:216) ~[classes/:?]
	at java.base/java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:539) [?:?]
	at java.base/java.util.concurrent.FutureTask.run$$$capture(FutureTask.java:264) [?:?]
	at java.base/java.util.concurrent.FutureTask.run(FutureTask.java) [?:?]
	at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:304) [?:?]
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) [?:?]
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) [?:?]
	at io.netty.util.concurrent.FastThreadLocalRunnable.run(FastThreadLocalRunnable.java:30) [netty-common-4.1.108.Final.jar:4.1.108.Final]
	at java.base/java.lang.Thread.run(Thread.java:833) [?:?]
Caused by: org.apache.zookeeper.KeeperException$NotEmptyException: KeeperErrorCode = Directory not empty for /managed-ledgers/p1

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@Technoboy- Technoboy- self-assigned this Apr 22, 2024
@Technoboy- Technoboy- added this to the 3.3.0 milestone Apr 22, 2024
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Apr 22, 2024
@Technoboy- Technoboy- requested a review from merlimat April 23, 2024 07:50
@merlimat merlimat merged commit d5c7231 into apache:master Apr 23, 2024
Technoboy- added a commit to Technoboy-/pulsar that referenced this pull request Apr 24, 2024
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request May 31, 2024
nikhil-ctds added a commit to datastax/pulsar that referenced this pull request Jun 3, 2024
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Jun 3, 2024
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Jun 4, 2024
mukesh154 pushed a commit to cognitree/ds_pulsar that referenced this pull request Jun 4, 2024
mukesh154 pushed a commit to cognitree/ds_pulsar that referenced this pull request Jun 5, 2024
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Jun 11, 2024
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Dec 27, 2024
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Dec 30, 2024
hanmz pushed a commit to hanmz/pulsar that referenced this pull request Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants