[
https://issues.apache.org/jira/browse/MRUNIT-188?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13800046#comment-13800046
]
Terry Koch edited comment on MRUNIT-188 at 10/20/13 9:48 PM:
-------------------------------------------------------------
Proposed fix:
1) Added a boolean to TestDriver to track whether a driver has been reused (ie
has executed run() more than once).
2) Added a check and set to preRunChecks() in each of the drivers to either
throw an exception if it is being reused or to set the boolean to true for the
first run().
Caveat - this patch still has my Javadoc alteration from JIRA 189 (sorry, my
bad, n00b mistake) in MapDriverBase, which also has the preRunChecks() addition
- I've since been taught how to avoid this in the future and will back out and
redo changes if that's an issue.
was (Author: terrykoch):
Proposed fix:
1) Added a boolean to TestDriver to track whether a driver has been reused (ie
has executed run() more than once).
2) Added a check and set to preRunChecks() in each of the drivers to either
throw an exception if it is being reused or to set the boolean to true for the
first run().
Caveat - this patch still has my Javadoc alteration from JIRA 188 (sorry, my
bad, n00b mistake) in MapDriverBase, which also has the preRunChecks() addition
- I've since been taught how to avoid this in the future but will back out and
redo changes if that's an issue.d
> Throw assertion error if drivers are reused
> -------------------------------------------
>
> Key: MRUNIT-188
> URL: https://issues.apache.org/jira/browse/MRUNIT-188
> Project: MRUnit
> Issue Type: Bug
> Reporter: Brock Noland
> Assignee: Terry Koch
> Priority: Blocker
> Fix For: 1.1.0
>
> Attachments: MRUNIT-188_v1.patch
>
>
> We should throw an assertion error if a driver.run method is called more than
> once. See MRUNIT-187 for more details.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
