You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/10/11 06:13:05 UTC

[GitHub] [pulsar] Technoboy- opened a new pull request, #17993: Fix flaky test `testSplitBundleForMultiTimes`.

Technoboy- opened a new pull request, #17993:
URL: https://github.com/apache/pulsar/pull/17993

   ### Motivation
   
   ```
   Error:  Tests run: 52, Failures: 1, Errors: 0, Skipped: 27, Time elapsed: 353.204 s <<< FAILURE! - in org.apache.pulsar.broker.admin.NamespacesTest
     Error:  testSplitBundleForMultiTimes(org.apache.pulsar.broker.admin.NamespacesTest)  Time elapsed: 0.326 s  <<< FAILURE!
     org.apache.pulsar.client.admin.PulsarAdminException$ServerSideErrorException: 
     
      --- An unexpected error occurred in the server ---
     
     Message: Namespace bundle my-tenant/namespace-86da66f1-e68d-480a-b5bb-9f78a2f52fc9/0x00000000_0x08000000 is being unloaded
     
     Stacktrace:
     
     java.lang.IllegalStateException: Namespace bundle my-tenant/namespace-86da66f1-e68d-480a-b5bb-9f78a2f52fc9/0x00000000_0x08000000 is being unloaded
     	at org.apache.pulsar.broker.namespace.NamespaceService.lambda$findBrokerServiceUrl$9(NamespaceService.java:419)
   ```
   
   https://github.com/apache/pulsar/actions/runs/3216911154/jobs/5275209470
   
   ### Modifications
   
   <!-- Describe the modifications you've done. -->
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   
   *(Please pick either of the following options)*
   
   This change is a trivial rework / code cleanup without any test coverage.
   
   *(or)*
   
   This change is already covered by existing tests, such as *(please describe tests)*.
   
   *(or)*
   
   This change added tests and can be verified as follows:
   
   *(example:)*
     - *Added integration tests for end-to-end deployment with large payloads (10MB)*
     - *Extended integration test for recovery after broker failure*
   
   ### Does this pull request potentially affect one of the following parts:
   
   *If the box was checked, please highlight the changes*
   
   - [ ] Dependencies (add or upgrade a dependency)
   - [ ] The public API
   - [ ] The schema
   - [ ] The default values of configurations
   - [ ] The binary protocol
   - [ ] The REST endpoints
   - [ ] The admin CLI options
   - [ ] Anything that affects deployment
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes -->
   - [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
   - [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
   - [ ] `doc-complete` <!-- Docs have been already added -->
   
   ### Matching PR in forked repository
   
   PR in forked repository: https://github.com/Technoboy-/pulsar/pull/10
   


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] Technoboy- commented on pull request #17993: [fix][test] Fix flaky test `testSplitBundleForMultiTimes`.

Posted by GitBox <gi...@apache.org>.
Technoboy- commented on PR #17993:
URL: https://github.com/apache/pulsar/pull/17993#issuecomment-1275495610

   > LGTM, but I'm considering if some PRs break this test. have you seen it before? Does it relate to this PR? #17797
   
   yes, right. I will update it to the motivation.


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] Technoboy- closed pull request #17993: [fix][test] Fix flaky test `testSplitBundleForMultiTimes`.

Posted by GitBox <gi...@apache.org>.
Technoboy- closed pull request #17993: [fix][test] Fix flaky test `testSplitBundleForMultiTimes`.
URL: https://github.com/apache/pulsar/pull/17993


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] Technoboy- merged pull request #17993: [fix][test] Fix flaky test `testSplitBundleForMultiTimes`.

Posted by GitBox <gi...@apache.org>.
Technoboy- merged PR #17993:
URL: https://github.com/apache/pulsar/pull/17993


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org