abstractdog commented on PR #367:
URL: https://github.com/apache/tez/pull/367#issuecomment-2538451426
thanks a lot @zhengchenyu for taking care of this
I believe this is almost ready to go in, let me ask one more thing
so I just confirmed that the unit patch indeed reproduces the issue, which
remains solved with reverting TEZ-4542 + applying this(long) cast
what is strange is without the patch, the full trace of the
IllegalArgumentException is not visible, I can only see:
```
[ERROR]
org.apache.tez.runtime.library.common.sort.impl.TestPipelinedSorter.testWithLargeRecordAndLowMemory
Time elapsed: 1.584 s <<< ERROR!
java.lang.IllegalArgumentException
at
org.apache.tez.runtime.library.common.sort.impl.TestPipelinedSorter.testWithLargeRecordAndLowMemory(TestPipelinedSorter.java:878)
[INFO]
[INFO] Results:
```
can you please check if this can be easily solved in the scope of this
patch? (or can you see the same on your machine?)
thanks in advance!
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
