[#992] fix(tez): convertTaskAttemptIdToLong should not consider appattemptId

[zhengchenyu]

What changes were proposed in this pull request?

When we convert task attempt id to long, consider real taskAttemptID = taskAttemptID - (appAttemptId - 1) * 1000, the logical only apply to mr. tez should remove this.

If appAttemptId is 2, taskattemptid will be normalized to negative value, it is unexpected.

Fix: #992

[jerqi]

LGTM, thanks @zhengchenyu

[codecov-commenter]

Codecov Report

Merging #1007 (3575e8d) into master (411b445) will increase coverage by 1.16%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##             master    #1007      +/-   ##
============================================
+ Coverage     53.67%   54.83%   +1.16%     
  Complexity     2523     2523              
============================================
  Files           381      361      -20     
  Lines         21648    19284    -2364     
  Branches       1800     1799       -1     
============================================
- Hits          11620    10575    -1045     
+ Misses         9322     8078    -1244     
+ Partials        706      631      -75     

Impacted Files	Coverage Δ
...c/main/java/org/apache/tez/common/RssTezUtils.java	63.13% <50.00%> (+0.57%)	⬆️
...ez/runtime/library/common/sort/impl/RssSorter.java	57.95% <100.00%> (-0.48%)	⬇️
.../runtime/library/common/sort/impl/RssUnSorter.java	63.63% <100.00%> (-0.41%)	⬇️

... and 25 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more