mridulm commented on a change in pull request #35185:
URL: https://github.com/apache/spark/pull/35185#discussion_r786339891
##########
File path:
core/src/test/resources/HistoryServerExpectations/excludeOnFailure_for_stage_expectation.json
##########
@@ -631,6 +642,7 @@
"taskId" : 4,
"index" : 2,
"attempt" : 1,
+ "partitionId": -1,
"launchTime" : "2018-01-09T10:21:18.943GMT",
"duration" : 16,
Review comment:
Sounds good to keeping it as-is.
We need to validate that missing value defaults to -1 (for both json and
level db) - to test upgrade case (this is dependent on Dongjoon's suggestions
above on how to mitigate the issue though).
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
