NUTCH-3057 - Fix for index-arbitrary plugin improper retention and us…

[CatChullain]

Fix for NUTCH-3057 where index-arbitrary plugin retained value for a field and erroneously set it to the next field declared in its config stanzas

[lewismc]

Thanks for reporting @CatChullain i didn’t catch this edge case either when reviewing or testing.
Out curiosity what does your deployment look like? Local or deploy?

[lewismc]

Hi @CatChullain please send an email to me directly at lewismc [at] apache [dot] org. We would like to invite to to become a committer to the Nutch project.
Thank you

[sebastian-nagel]

+1 lgtm.

[lewismc]

+1 please merge @CatChullain

[CatChullain]

Thanks!

Merged now and resolved the Jira. Delete the fix's branch after the merge? Or wait till after the next public release?

[sebastian-nagel]

Delete the fix's branch after the merge? Or wait till after the next public release?

Even if you delete the underlying branch, all changes and commit in the PR are still visible. So, it's safe to delete the branch now. Otherwise: since the branch is on your repository, it's solely your decision.

[CatChullain]

Thanks! Yeah, I see now that it says it's on my repo. I was thinking it was in the project's.