[
https://issues.apache.org/jira/browse/OOZIE-2231?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15423904#comment-15423904
]
Robert Kanter commented on OOZIE-2231:
--------------------------------------
LGTM, a few minor things:
# The change to {{JvmPauseMonitorService}} seems unrelated. We should leave
that code alone.
# The indentation for {{<guava.version>}} is off by one.
# Instead of putting the pig guava version number in the pig pom, let's do
something similar to what we did with {{<spark.guava.version>}}, and create a
{{<pig.guava.version>}}.
# This updates Guava to 16. Hadoop currently requires Guava 11, which I
believe is therefore required by Oozie because of the Hadoop dependencies in
Oozie. Have you actually tested that this isn't a problem? (Hadoop 3 should
address this problem with classpath isolation)
> Upgrade curator to latest version 2.8.0
> ---------------------------------------
>
> Key: OOZIE-2231
> URL: https://issues.apache.org/jira/browse/OOZIE-2231
> Project: Oozie
> Issue Type: Bug
> Components: HA
> Reporter: Purshotam Shah
> Assignee: Abhishek Bafna
> Fix For: 4.3.0
>
> Attachments: OOZIE-2231-00.patch, OOZIE-2231-01.patch
>
>
> It have some fix related to InterProcessReadWriteLock, ChildReaper,
> LeaderSelector which we use.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
