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/09/08 10:11:32 UTC

[GitHub] [pulsar] poorbarcode opened a new pull request, #17546: [do not merge][fix][flaky-test]PulsarFunctionTlsTest.testFunctionsCreation

poorbarcode opened a new pull request, #17546:
URL: https://github.com/apache/pulsar/pull/17546

   Fixes:
   
   - #17112
   
   ### Motivation
   
   Add more context
   
   ### Modifications
   
   Add more context
   
   ### Documentation
   
   - [ ] `doc-required` 
     
   - [ ] `doc-not-needed` 
     
   - [ ] `doc` 
   
   - [ ] `doc-complete`


-- 
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] poorbarcode commented on pull request #17546: [fix][flaky-test]PulsarFunctionTlsTest.testFunctionsCreation

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

   Hi @lhotari 
   
   > PulsarFunctionTlsTest still gets stuck in tearDown, here's a recent stacktrace:
   
   Thank you. This information is very useful. I will try to find the root cause


-- 
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] lhotari commented on pull request #17546: [do not merge][fix][flaky-test]PulsarFunctionTlsTest.testFunctionsCreation

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

   @poorbarcode is this ready for merging?


-- 
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] poorbarcode commented on pull request #17546: [do not merge][fix][flaky-test]PulsarFunctionTlsTest.testFunctionsCreation

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

   Hi @lhotari 
   
   > is this ready for merging?
   
   Yes. I am writing `Motivation` to explain why this change, so labeled `[do not merge]`


-- 
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] lhotari commented on pull request #17546: [fix][flaky-test]PulsarFunctionTlsTest.testFunctionsCreation

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

   @poorbarcode I merged this. Thanks for making this change. you can write the motivation later.


-- 
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] poorbarcode commented on pull request #17546: [fix][flaky-test]PulsarFunctionTlsTest.testFunctionsCreation

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

   Hi @lhotari 
   
   > I merged this. Thanks for making this change. you can write the motivation later.
   
   Thanks, I have added the motivation, and I will continue to look up the root problems


-- 
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] lhotari merged pull request #17546: [fix][flaky-test]PulsarFunctionTlsTest.testFunctionsCreation

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


-- 
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] lhotari commented on pull request #17546: [fix][flaky-test]PulsarFunctionTlsTest.testFunctionsCreation

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

   PulsarFunctionTlsTest still gets stuck in tearDown, here's a recent stacktrace:
   ```
   "main" #1 prio=5 os_prio=0 cpu=43749.23ms elapsed=3506.78s tid=0x00007f965c026e90 nid=0xac2 waiting on condition  [0x00007f9660e26000]
      java.lang.Thread.State: WAITING (parking)
   	at jdk.internal.misc.Unsafe.park(java.base@17.0.4.1/Native Method)
   	- parking to wait for  <0x0000100032218608> (a java.util.concurrent.CompletableFuture$Signaller)
   	at java.util.concurrent.locks.LockSupport.park(java.base@17.0.4.1/LockSupport.java:211)
   	at java.util.concurrent.CompletableFuture$Signaller.block(java.base@17.0.4.1/CompletableFuture.java:1864)
   	at java.util.concurrent.ForkJoinPool.unmanagedBlock(java.base@17.0.4.1/ForkJoinPool.java:3463)
   	at java.util.concurrent.ForkJoinPool.managedBlock(java.base@17.0.4.1/ForkJoinPool.java:3434)
   	at java.util.concurrent.CompletableFuture.waitingGet(java.base@17.0.4.1/CompletableFuture.java:1898)
   	at java.util.concurrent.CompletableFuture.join(java.base@17.0.4.1/CompletableFuture.java:2117)
   	at org.apache.pulsar.metadata.coordination.impl.LockManagerImpl.close(LockManagerImpl.java:163)
   	at org.apache.pulsar.broker.loadbalance.impl.ModularLoadManagerImpl.stop(ModularLoadManagerImpl.java:983)
   	at org.apache.pulsar.broker.loadbalance.impl.ModularLoadManagerWrapper.stop(ModularLoadManagerWrapper.java:118)
   	at org.apache.pulsar.functions.worker.PulsarFunctionTlsTest.tearDown(PulsarFunctionTlsTest.java:193)
   ```
   in https://github.com/apache/pulsar/actions/runs/3044816917/jobs/4905919291#step:11:528


-- 
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