fix restTime bug

[CoffeeLatte007]

Ⅰ. Describe what this PR did

fix restTime bug

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

[codecov-io]

Codecov Report

Merging #954 into develop will decrease coverage by 0.06%.
The diff coverage is 100%.

@@              Coverage Diff              @@
##             develop     #954      +/-   ##
=============================================
- Coverage      37.17%   37.11%   -0.07%     
  Complexity      1027     1027              
=============================================
  Files            225      225              
  Lines           8906     8905       -1     
  Branches        1070     1070              
=============================================
- Hits            3311     3305       -6     
- Misses          5209     5212       +3     
- Partials         386      388       +2

Impacted Files	Coverage Δ	Complexity Δ
...in/java/io/seata/server/session/GlobalSession.java	84.35% <100%> (-1.76%)	50 <0> (ø)
...o/seata/server/coordinator/DefaultCoordinator.java	47.46% <0%> (-1.27%)	17% <0%> (ø)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f4469ea...c9deada. Read the comment docs.

[nikyotensai]

I think that the unit of restTime is wrong.Should change it to nanosecond?

[CoffeeLatte007]

I think that the unit of restTime is wrong.Should change it to nanosecond?

why? mills is enough

[nikyotensai]

I think that the unit of restTime is wrong.Should change it to nanosecond?

why? mills is enough

long parkTime = backOffTime < restTime ? backOffTime : restTime;
LockSupport.parkNanos(parkTime);

The unit between backOffTime and restTime is different.

[CoffeeLatte007]

restTime

long parkTime = backOffTime < restTime ? backOffTime : restTime;
LockSupport.parkNanos(parkTime);

The unit between backOffTime and restTime is different.

I think that the unit of restTime is wrong.Should change it to nanosecond?

why? mills is enough

long parkTime = backOffTime < restTime ? backOffTime : restTime;
LockSupport.parkNanos(parkTime);

The unit between backOffTime and restTime is different.

yes , i will fix it

[CoffeeLatte007]

I think that the unit of restTime is wrong.Should change it to nanosecond?

why? mills is enough

long parkTime = backOffTime < restTime ? backOffTime : restTime;
LockSupport.parkNanos(parkTime);

The unit between backOffTime and restTime is different.

i fix it, please review

[nikyotensai]

I think that the unit of restTime is wrong.Should change it to nanosecond?

why? mills is enough

long parkTime = backOffTime < restTime ? backOffTime : restTime;
LockSupport.parkNanos(parkTime);

The unit between backOffTime and restTime is different.

i fix it, please review

The PARK_TIMES_BASE will be 1000 times of its original value,I'm not sure whether it is right or not.

[CoffeeLatte007]

I think that the unit of restTime is wrong.Should change it to nanosecond?

why? mills is enough

long parkTime = backOffTime < restTime ? backOffTime : restTime;
LockSupport.parkNanos(parkTime);

The unit between backOffTime and restTime is different.

i fix it, please review

The PARK_TIMES_BASE will be 1000 times of its original value,I'm not sure whether it is right or not.

fix it PARK_TIMES_BASE_MILLS = 1

[nikyotensai]

LGTM

[slievrly]

LGTM

[xingfudeshi]

LGTM.