Skip to content

Conversation

@kishorvpatil
Copy link
Contributor

What is the purpose of the change

  • StormTimer threads are not names properly on Supervisor.

How was the change tested

Running unit test and jstack shows Thread names

   java.lang.Thread.State: TIMED_WAITING (sleeping)
	at java.lang.Thread.sleep(Native Method)
	at org.apache.storm.utils.Time.sleep(Time.java:100)
	at org.apache.storm.StormTimer$StormTimerTask.run(StormTimer.java:236)

"WorkerHBTimer" #21 daemon prio=10 os_prio=0 tid=0x00007f92cce62800 nid=0x58a2 waiting on condition [0x00007f92b5ad7000]
   java.lang.Thread.State: TIMED_WAITING (sleeping)
	at java.lang.Thread.sleep(Native Method)
	at org.apache.storm.utils.Time.sleep(Time.java:100)
	at org.apache.storm.StormTimer$StormTimerTask.run(StormTimer.java:236)

"HBTimer" #20 daemon prio=10 os_prio=0 tid=0x00007f92cce61000 nid=0x58a1 waiting on condition [0x00007f92b5bd8000]
   java.lang.Thread.State: TIMED_WAITING (sleeping)
	at java.lang.Thread.sleep(Native Method)
	at org.apache.storm.utils.Time.sleep(Time.java:100)
	at org.apache.storm.StormTimer$StormTimerTask.run(StormTimer.java:236)

@agresch agresch merged commit e832204 into apache:master Jun 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants