Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-02 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21046/ --- Review request for oozie. Bugs: OOZIE-1715 https://issues.apache.org/jira/b

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-05 Thread Robert Kanter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21046/#review42223 --- core/src/main/java/org/apache/oozie/service/DistributedUUIDService.

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-13 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21046/ --- (Updated May 13, 2014, 5:32 p.m.) Review request for oozie. Bugs: OOZIE-1715

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-13 Thread Purshotam Shah
> On May 6, 2014, 12:23 a.m., Robert Kanter wrote: > > Thanks, for reviewing. We also added option to rest sequence once it reach limit. > On May 6, 2014, 12:23 a.m., Robert Kanter wrote: > > core/src/main/java/org/apache/oozie/service/DistributedUUIDService.java, > > line 44 > >

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-13 Thread Robert Kanter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21046/#review42891 --- ID's usually look like this: 000-140505141334644-oozie-oozi-W wh

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-16 Thread Purshotam Shah
> On May 13, 2014, 9:55 p.m., Robert Kanter wrote: > > ID's usually look like this: 000-140505141334644-oozie-oozi-W > > where the second value is a timestamp and the first value is the increasing > > part. > > > > With the ZKUUIDService, will all Oozie servers use the same timestamp part

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-16 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21046/ --- (Updated May 15, 2014, 6:43 p.m.) Review request for oozie. Bugs: OOZIE-1715

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-19 Thread Robert Kanter
> On May 13, 2014, 9:55 p.m., Robert Kanter wrote: > > core/src/main/java/org/apache/oozie/service/ZKUUIDService.java, line 55 > > > > > > I'm not sure we should make the path configurable; I'm concerned that > > will

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-19 Thread Robert Kanter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21046/#review43396 --- Can you add the new config properties to oozie-default and/or oozie-

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-19 Thread Purshotam Shah
> On May 19, 2014, 6:57 p.m., Robert Kanter wrote: > > core/src/main/java/org/apache/oozie/service/ZKUUIDService.java, line 60 > > > > > > What happens if the user sets this too high (i.e. more than the number > > of d

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-05-19 Thread Purshotam Shah
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21046/ --- (Updated May 20, 2014, 12:47 a.m.) Review request for oozie. Bugs: OOZIE-1715

Re: Review Request 21046: OOZIE-1715 Distributed ID sequence for HA

2014-06-02 Thread Robert Kanter
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/21046/#review44575 --- Ship it! Ship It! - Robert Kanter On May 20, 2014, 12:47 a.m., P