Re: Review Request 68317: Sqoop Action should support tez delegation tokens for hive-import

> On febr. 18, 2019, 8:43 du, Peter Bacsko wrote: > > sharelib/oozie/src/main/java/org/apache/oozie/action/hadoop/LauncherAM.java > > Lines 94-96 (patched) > > > > > > I think we already support Java8, so it can be a

Re: Review Request 68317: Sqoop Action should support tez delegation tokens for hive-import

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68317/#review212905 ---

Re: Review Request 69251: OOZIE-3377 [docs] Remaining 5.1.0 documentation changes

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69251/#review210321 --- Ship it! Ship It! - Peter Bacsko On nov. 5, 2018, dél,

Re: Review Request 68909: amend OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68909/#review209182 --- Ship it! Ship It! - Peter Bacsko On okt. 3, 2018, 9:43 de,

Re: Review Request 68737: OOZIE-3307 [core][oya] Limit heap usage of LauncherAM

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68737/#review208948 --- Ship it! Ship It! - Peter Bacsko On szept. 24, 2018, 3:17

Re: Review Request 68737: OOZIE-3307 [core][oya] Limit heap usage of LauncherAM

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68737/#review208799 ---

Re: Review Request 68737: OOZIE-3307 [core][oya] Limit heap usage of LauncherAM

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68737/#review208750 ---

Re: Review Request 68737: OOZIE-3307 [core][oya] Limit heap usage of LauncherAM

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68737/#review208705 ---

Re: Review Request 68237: OOZIE-3298 OYA: external ID is not filled properly and failing MR job is treated as SUCCEEDED

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68237/#review208483 --- Ship it! Ship It! - Peter Bacsko On szept. 7, 2018, 8:08

Re: Review Request 68237: OOZIE-3298 OYA: external ID is not filled properly and failing MR job is treated as SUCCEEDED

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68237/#review208426 ---

Re: Review Request 67885: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated szept. 5, 2018, 11:59 de) Review request for oozie, András Piros,

Re: Review Request 67885: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated szept. 5, 2018, 11:58 de) Review request for oozie, András Piros,

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated szept. 4, 2018, 3:45 du) Review request for oozie, András Piros,

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated szept. 4, 2018, 3:42 du) Review request for oozie, András Piros,

Re: Review Request 68505: OOZIE-2877 Git action

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68505/#review208306 --- Ship it! Ship It! - Peter Bacsko On aug. 31, 2018, 9:50 de,

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated szept. 4, 2018, 12:05 du) Review request for oozie, András Piros,

Re: Review Request 68237: OOZIE-3298 OYA: external ID is not filled properly and failing MR job is treated as SUCCEEDED

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68237/#review208130 ---

Re: Review Request 68505: OOZIE-2877 Git action

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68505/#review208091 ---

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated aug. 17, 2018, 12:37 du) Review request for oozie, András Piros,

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated aug. 17, 2018, 12:36 du) Review request for oozie, András Piros,

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated aug. 16, 2018, 3:55 du) Review request for oozie, András Piros, Peter

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/#review207318 ---

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

> On aug. 10, 2018, 2:44 du, András Piros wrote: > > core/src/main/java/org/apache/oozie/service/AsyncXCommandExecutor.java > > Lines 143 (patched) > > > > > > What about using a `CopyOnWriteArrayList` or a

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated aug. 15, 2018, 9:49 de) Review request for oozie, András Piros, Peter

Re: Review Request 59620: This review board request is for an action to provide a Git action for Oozie

> On júl. 2, 2018, 12:46 du, Peter Bacsko wrote: > > sharelib/git/src/main/java/org/apache/oozie/action/hadoop/GitMain.java > > Lines 41 (patched) > > > > > > We don't use XLog inside LauncherMain, only Sysout. This

Re: Review Request 59620: This review board request is for an action to provide a Git action for Oozie

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59620/#review206974 ---

Re: Review Request 59620: This review board request is for an action to provide a Git action for Oozie

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59620/#review206939 ---

Re: Review Request 59620: This review board request is for an action to provide a Git action for Oozie

> On júl. 2, 2018, 12:46 du, Peter Bacsko wrote: > > docs/src/site/twiki/WorkflowFunctionalSpec.twiki > > Lines 1667-1669 (patched) > > > > > > Do we really need to access a different cluster other than the one >

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated aug. 6, 2018, 3:28 du) Review request for oozie, András Piros, Peter

Re: Review Request 67999: Parsing sharelib timestamps is not threadsafe

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67999/#review206798 ---

Re: Review Request 68140: OOZIE-3229 Improved filtering options in V2SLAServlet

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/68140/#review206747 --- I haven't written specific comments yet, but in general, there

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated júl. 31, 2018, 11 de) Review request for oozie, András Piros and

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated júl. 13, 2018, 10:05 de) Review request for oozie, András Piros and

Re: Review Request 67885: POC: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67885/ --- (Updated júl. 13, 2018, 10:03 de) Review request for oozie, András Piros and

Re: Review Request 66930: OOZIE-2791 ShareLib installation may fail on busy Hadoop clusters

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66930/#review205714 --- Ship it! Ship It! - Peter Bacsko On júl. 3, 2018, 3:24 du,

Re: Review Request 66930: OOZIE-2791 ShareLib installation may fail on busy Hadoop clusters

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66930/#review205672 --- tools/src/main/java/org/apache/oozie/tools/OozieSharelibCLI.java

Re: Review Request 66930: OOZIE-2791 ShareLib installation may fail on busy Hadoop clusters

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66930/#review205638 --- tools/src/main/java/org/apache/oozie/tools/OozieSharelibCLI.java

Re: Review Request 59620: This review board request is for an action to provide a Git action for Oozie

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59620/#review205637 ---

Re: Review Request 59620: This review board request is for an action to provide a Git action for Oozie

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59620/#review205636 ---

Re: Review Request 64029: OOZIE-2339 Provide an API for writing jobs based on the XSD schemas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64029/#review204771 ---

Re: Review Request 64029: OOZIE-2339 Provide an API for writing jobs based on the XSD schemas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64029/#review204692 ---

Re: Review Request 66930: OOZIE-2791 ShareLib installation may fail on busy Hadoop clusters

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/66930/#review204598 --- tools/src/main/java/org/apache/oozie/tools/OozieSharelibCLI.java

Re: Review Request 67284: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67284/ --- (Updated máj. 24, 2018, 3:29 du) Review request for oozie, András Piros, Peter

Review Request 67284: OOZIE-3160 PriorityDelayQueue put()/take() can cause significant CPU load due to busy waiting

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67284/ --- Review request for oozie, András Piros, Peter Cseh, and Robert Kanter.

Re: Review Request 67183: OOZIE-3237 Flaky test TestZKLocksService#testWriteReadLockThreads

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67183/ --- (Updated máj. 22, 2018, 3:49 du) Review request for oozie, András Piros and

Re: Review Request 67183: OOZIE-3237 Flaky test TestZKLocksService#testWriteReadLockThreads

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67183/#review203570 --- core/src/test/java/org/apache/oozie/lock/TestMemoryLocks.java

Re: Review Request 67183: OOZIE-3237 Flaky test TestZKLocksService#testWriteReadLockThreads

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67183/#review203330 --- core/src/test/java/org/apache/oozie/util/LatchHandler.java Lines

Review Request 67183: OOZIE-3237 Flaky test TestZKLocksService#testWriteReadLockThreads

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67183/ --- Review request for oozie, András Piros and Peter Cseh. Repository: oozie-git

Review Request 67133: OOZIE-3236 Flaky test TestHiveActionExecutor#testHiveAction

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67133/ --- Review request for oozie, András Piros and Peter Cseh. Repository: oozie-git

Review Request 67132: OOZIE-3246 Flaky test TestJMSJobEventListener#testConnectionDrop

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67132/ --- Review request for oozie, András Piros and Peter Cseh. Repository: oozie-git

Re: Review Request 67129: OOZIE-3240 Flaky test TestJMSAccessorService#testConnectionRetry

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67129/ --- (Updated máj. 15, 2018, 12:52 du) Review request for oozie, András Piros and

Re: Review Request 67129: OOZIE-3240 Flaky test TestJMSAccessorService#testConnectionRetry

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67129/#review203109 ---

Review Request 67129: OOZIE-3240 Flaky test TestJMSAccessorService#testConnectionRetry

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67129/ --- Review request for oozie, András Piros and Peter Cseh. Repository: oozie-git

Re: Review Request 65606: OOZIE-2814 OYA: Update example workflows to newest schemas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65606/#review197501 ---

Re: Review Request 65606: OOZIE-2814 OYA: Update example workflows to newest schemas

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65606/#review197500 ---

Re: Review Request 65481: OOZIE-3134 - Potential inconsistency between the in-memory SLA map and the Oozie database

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65481/#review196901 ---

Re: Review Request 65287: OOZIE-3157 - Setup truststore so that it also works in HTTP only mode

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65287/#review196114 ---

Re: Review Request 65177: >git status< should be clean after >mvn test< was called

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65177/#review195568 --- core/src/test/java/org/apache/oozie/test/XTestCase.java Lines

Re: Review Request 64408: OOZIE-2869 amend Ability to add/remove MapReduce jars from action classpath

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64408/#review193130 ---

Re: Review Request 64408: OOZIE-2869 amend Ability to add/remove MapReduce jars from action classpath

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64408/#review193132 ---

Re: Review Request 64245: OOZIE-2681 fix javadoc to compile on JDK8 again

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64245/#review192496 --- core/src/main/java/org/apache/oozie/CoordinatorEngine.java Lines

Re: Review Request 63875: OOZIE-2900 Retrieve tokens for oozie.launcher.mapreduce.job.hdfs-servers before submission

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63875/#review191831 --- Ship it! Ship It! - Peter Bacsko On nov. 24, 2017, 10:29

Re: Review Request 63875: OOZIE-2900 Retrieve tokens for oozie.launcher.mapreduce.job.hdfs-servers before submission

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63875/#review191802 ---

Re: Review Request 63875: OOZIE-2900 Retrieve tokens for oozie.launcher.mapreduce.job.hdfs-servers before submission

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63875/#review191343 ---

Re: Review Request 63327: OOZIE-2897 LauncherAM should support ACLs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63327/ --- (Updated nov. 7, 2017, 12:36 du) Review request for oozie, András Piros,

Re: Review Request 63327: OOZIE-2897 LauncherAM should support ACLs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63327/ --- (Updated nov. 6, 2017, 12:08 du) Review request for oozie, András Piros,

Re: Review Request 63327: OOZIE-2897 LauncherAM should support ACLs

> On nov. 3, 2017, 10:33 de, András Piros wrote: > > core/src/test/java/org/apache/oozie/test/XTestCase.java > > Lines 1046-1047 (patched) > > > > > > Depending on the runtime value of > >

Re: Review Request 63327: OOZIE-2897 LauncherAM should support ACLs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63327/#review189721 ---

Re: Review Request 63327: OOZIE-2897 LauncherAM should support ACLs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63327/ --- (Updated okt. 31, 2017, 3:02 du) Review request for oozie, András Piros,

Re: Review Request 63327: OOZIE-2897 LauncherAM should support ACLs

> On okt. 27, 2017, 1:47 du, András Piros wrote: > > core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java > > Lines 1659-1687 (patched) > > > > > > In general, that's a good negative test.

Re: Review Request 63327: OOZIE-2897 LauncherAM should support ACLs

> On okt. 27, 2017, 1:47 du, András Piros wrote: > > core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java > > Lines 154-155 (patched) > > > > > > Should go to

Re: Review Request 62936: OOZIE-2896 Ensure compatibility for existing LauncherMapper settings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62936/#review189417 --- Ship it! - Peter Bacsko On okt. 27, 2017, 10:01 de, András

Review Request 63327: OOZIE-2897 LauncherAM should support ACLs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63327/ --- Review request for oozie, András Piros, Attila Sasvari, Peter Cseh, and Robert

Re: Review Request 62936: OOZIE-2896 Ensure compatibility for existing LauncherMapper settings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62936/#review188680 ---

Re: Review Request 62936: OOZIE-2896 Ensure compatibility for existing LauncherMapper settings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62936/#review188678 ---

Re: Review Request 62936: OOZIE-2896 Ensure compatibility for existing LauncherMapper settings

> On okt. 17, 2017, 3:19 du, András Piros wrote: > > core/src/main/java/org/apache/oozie/action/hadoop/LauncherConfigurationInjector.java > > Lines 95-96 (patched) > > > > > > I'm really rather in favor of making

Re: Review Request 62936: OOZIE-2896 Ensure compatibility for existing LauncherMapper settings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62936/#review188290 ---

Re: Review Request 62936: OOZIE-2896 Ensure compatibility for existing LauncherMapper settings

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62936/#review188120 ---