Re: Review Request 52373: OOZIE-2688 YAML representation of Oozie unit testcase

2017-08-04 Thread Purshotam Shah via Review Board 

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

(Updated Aug. 5, 2017, 1:03 a.m.)


Review request for oozie.


Bugs: OOZIE-2688
https://issues.apache.org/jira/browse/OOZIE-2688


Repository: oozie-git


Description
---

Oozie YAML Unit test

Maven configuration
YAML configuration
Declaration
Import
setup
Initialization
Job Definition
HDFS operation
HCat operation
Job stubbing
Usage Guideline
validation
output validation
job validation
Usage Guideline
TearUp
Tag
Examples
Bundle Test
Coordinator test
Coordinator test with HCat
Workflow Test
Workflow Test with skip Job submission
API
Maven configuration

To enable oozie unit yaml, one need add oozie-yaml plugin to maven.


org.apache.oozie
oozie-yaml
0.1-SNAPSHOT



oozieyaml




Oozie-YAML plugin to Maven will search for any oozie YAML file and if it's 
there, it will convert it into respective JUnit test case and will execute 
them. There will be a surefire report.
YAML configuration

Whole YAML configuration is divided into following sections

Declaration
Import
Setup
Initialization
Job Definition
Stubbing
HDFS operation
HCat operation
Validation
TearUp
Tag
Declaration

This section can be used to defined test case properties like test case name, 
package name, etc.

name: String, required
Name of the test cases. Java file will be created with the same name.
package: String, Optional
The Java package of the generated test class.
skip_test : boolean, Optional
A boolean that indicates whether or not to execute this test. This parameter 
defaults to false.
copy_dependencies_files_hdfs: List of Strings, Optional
List of HDFS of paths where all Maven dependent jar will be copied. This is 
useful if for pig/hive jobs.
Examples:

package: org.apache.oozie.yamltest
name: TestWorkflowActionFromBundle
copy_dependencies_files_hdfs:
   - /tmp/workflow-test/lib
Import

imports: List of Strings, Optional
A list of Java classes to import in the generated Oozie unit Java file. * 
Examples: *
 import:
  - org.apache.oozie.util.DateUtils
  - java.util.Date
setup

setup: List of Strings, Optional
List of java statement that can be used for JUnit setup method.
setup can also include HDFS and HCat operation.
Initialization

initialization: List of Strings, Optional
List of java statement that can be used for initialization. This statement will 
be added first in test case.
Examples:

initialization:
  - "Date date = new Date()"
  - "String currentTime = DateUtils.formatDateOozieTZ(date)"
Job Definition

This section can be used to define job properties

job_property: String, Optional
job_property can be a Java Properties file (.properties) or a Hadoop XML 
Configuration file (.xml).
properties: String, Optional
Overridden key/value configuration use in job submission. This value overrides 
the value of job_property.
Examples:

job:
properties:
raw_logs_path: "/tmp/test"
aggregated_logs_path: "/user/test"
workflowPath: "/testDirPath"
start: "2010-02-01T10:00Z"
job:
job_property : src/test/resources/job-1.properties
properties:
oozie.wf.application.path: "/tmp/workflow-test/"
jobTracker: !func getJobTrackerUri()
nameNode: !func getNameNodeUri()
HDFS operation

Set of HDFS operation. Can be used to create dummy input dataset for 
coordinator or input file for workflow. Supported operation are

mkdir: List, Optional
The mkdir command creates the specified directory. If the directory already 
exists, it does a no-op.
touchz: List, Optional
Touchz creates a zero length file in the specified path if none exists. If one 
already exists, then touchz will perform a touch operation.
delete: List, Optional
List of file/dir to be deleted.
delete: List, Optional
Deletes the specified path.
chmod: List, Optional
The chmod command changes the permissions for the specified path.
permission: String, Optional
Permission of path. Permissions can be specified using the Unix Symbolic 
representation (e.g. -rwxrw-rw-) or an octal representation (755).
path: String, Optional
HDFS path of file/dir whose permission need to changed.
copyFromLocal: List, Optional
List of file to be copied to HDFS
source: String, mandatory
destination: String, mandatory
move: List, Optional
source: String, mandatory
destination: String, mandatory
Examples:

hdfs:
touchz: 
  - /tmp/test/2010/02/01/09/_SUCCESS
copyFromLocal:
  - source: coordinator-test-3.xml
destination: /tmp/workflow-test/coordinator-test-3.xml
HCat operation

Set of HCat operation. Can be used to create HCat partition for 
coordinator/workflow. HCat operation can be configured in nested form. At 
parent level, at has two operation delete and create

create: Can be used to create HCat

Re: Review Request 58406: OOZIE-2863 SLACalculatorMemory.loadOnRestart causing delay in server start

2017-04-18 Thread Purshotam Shah via Review Board 

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

(Updated April 19, 2017, 1:23 a.m.)


Review request for oozie.


Bugs: OOZIE-2863
https://issues.apache.org/jira/browse/OOZIE-2863


Repository: oozie-git


Description
---

SLACalculatorMemory.loadOnRestart causing delay in server start


Diffs (updated)
-

  core/src/main/java/org/apache/oozie/sla/SLACalcStatus.java 3a76dfe6 
  core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java 347f8537 
  core/src/main/java/org/apache/oozie/sla/service/SLAService.java 08cd07e7 
  core/src/test/java/org/apache/oozie/service/TestHASLAService.java 3af263e1 
  core/src/test/java/org/apache/oozie/sla/TestSLACalculatorMemory.java 559e2b31 


Diff: https://reviews.apache.org/r/58406/diff/4/

Changes: https://reviews.apache.org/r/58406/diff/3-4/


Testing
---


Thanks,

Purshotam Shah

Re: Review Request 58406: OOZIE-2863 SLACalculatorMemory.loadOnRestart causing delay in server start

2017-04-18 Thread Purshotam Shah via Review Board 

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

(Updated April 18, 2017, 7 p.m.)


Review request for oozie.


Bugs: OOZIE-2863
https://issues.apache.org/jira/browse/OOZIE-2863


Repository: oozie-git


Description
---

SLACalculatorMemory.loadOnRestart causing delay in server start


Diffs (updated)
-

  core/src/main/java/org/apache/oozie/sla/SLACalcStatus.java 3a76dfe6 
  core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java 347f8537 
  core/src/test/java/org/apache/oozie/service/TestHASLAService.java 3af263e1 
  core/src/test/java/org/apache/oozie/sla/TestSLACalculatorMemory.java 559e2b31 


Diff: https://reviews.apache.org/r/58406/diff/3/

Changes: https://reviews.apache.org/r/58406/diff/2-3/


Testing
---


Thanks,

Purshotam Shah

Re: Review Request 58406: OOZIE-2863 SLACalculatorMemory.loadOnRestart causing delay in server start

2017-04-18 Thread Purshotam Shah via Review Board 

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

(Updated April 18, 2017, 5:34 p.m.)


Review request for oozie.


Bugs: OOZIE-2863
https://issues.apache.org/jira/browse/OOZIE-2863


Repository: oozie-git


Description
---

SLACalculatorMemory.loadOnRestart causing delay in server start


Diffs (updated)
-

  core/src/main/java/org/apache/oozie/sla/SLACalcStatus.java 3a76dfe6 
  core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java 347f8537 
  core/src/test/java/org/apache/oozie/service/TestHASLAService.java 3af263e1 
  core/src/test/java/org/apache/oozie/sla/TestSLACalculatorMemory.java 559e2b31 


Diff: https://reviews.apache.org/r/58406/diff/2/

Changes: https://reviews.apache.org/r/58406/diff/1-2/


Testing
---


Thanks,

Purshotam Shah

Re: Review Request 58406: OOZIE-2863 SLACalculatorMemory.loadOnRestart causing delay in server start

2017-04-18 Thread Purshotam Shah via Review Board 


> On April 17, 2017, 11:42 p.m., Satish Saley wrote:
> > core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java
> > Lines 221 (patched)
> > 
> >
> > Please move this inside synchronized block on line below to get 
> > consistent view of slaCalc object across threads. Adjust the code inside 
> > checkAndUpdateSLACalcAfterRestart-->updateSLARegistartion accordingly. We 
> > don't need the synchronized block in updateSLARegistartion then.

We need to synchronize block so that only one thread update the slaCalc.


> On April 17, 2017, 11:42 p.m., Satish Saley wrote:
> > core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java
> > Line 250 (original), 234 (patched)
> > 
> >
> > Do we need else condition similar to that line 265?
> > 
> > ```
> > else{
> >  if (firstCheckAfterRetstart) {
> >slaCalc.setSLARegistrationBean(null);
> > }
> > }
> > ```

Not sure what do you mean by this?  Are you saying that we need to check 
following condition?

if (e.getErrorCode().equals(ErrorCode.E0604) || 
e.getErrorCode().equals(ErrorCode.E0605)) {
LOG.debug("job [{0}] is is not in DB, removing from 
Memory", slaCalc.getId());
slaMap.remove(jobId);
}


If yes, No we don't have to check if an object exists or not. addJobStatus will 
get called when there is a change in job status which means the object is in DB.


> On April 17, 2017, 11:42 p.m., Satish Saley wrote:
> > core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java
> > Line 476 (original), 464 (patched)
> > 
> >
> > Need to wrap the call to checkAndUpdateSLACalcAfterRestart in 
> > synchronized(slaCalc) {} with the changes I suggested regarding 
> > synchronized block on line 221.

If I understand correctly, Do you mean to say that we should synchronize 
checkAndUpdateSLACalcAfterRestart.

We don't want to synchronized check. Only update need synchronization.


> On April 17, 2017, 11:42 p.m., Satish Saley wrote:
> > core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java
> > Lines 657 (patched)
> > 
> >
> > No need to check slaCalc.getSLARegistrationBean() == null here, inside 
> > synchronized(slaCalc) we are doing it.

Yes. If the call fails, we need to make sure that next run will execute SLA 
check.


- Purshotam


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


On April 14, 2017, 10:41 p.m., Purshotam Shah wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58406/
> ---
> 
> (Updated April 14, 2017, 10:41 p.m.)
> 
> 
> Review request for oozie.
> 
> 
> Bugs: OOZIE-2863
> https://issues.apache.org/jira/browse/OOZIE-2863
> 
> 
> Repository: oozie-git
> 
> 
> Description
> ---
> 
> SLACalculatorMemory.loadOnRestart causing delay in server start
> 
> 
> Diffs
> -
> 
>   core/src/main/java/org/apache/oozie/sla/SLACalcStatus.java 3a76dfe6 
>   core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java 347f8537 
>   core/src/test/java/org/apache/oozie/service/TestHASLAService.java 3af263e1 
>   core/src/test/java/org/apache/oozie/sla/TestSLACalculatorMemory.java 
> 559e2b31 
> 
> 
> Diff: https://reviews.apache.org/r/58406/diff/1/
> 
> 
> Testing
> ---
> 
> 
> Thanks,
> 
> Purshotam Shah
> 
>

Review Request 58406: OOZIE-2863 SLACalculatorMemory.loadOnRestart causing delay in server start

2017-04-14 Thread Purshotam Shah via Review Board 

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

Review request for oozie.


Bugs: OOZIE-2863
https://issues.apache.org/jira/browse/OOZIE-2863


Repository: oozie-git


Description
---

SLACalculatorMemory.loadOnRestart causing delay in server start


Diffs
-

  core/src/main/java/org/apache/oozie/sla/SLACalcStatus.java 3a76dfe6 
  core/src/main/java/org/apache/oozie/sla/SLACalculatorMemory.java 347f8537 
  core/src/test/java/org/apache/oozie/service/TestHASLAService.java 3af263e1 
  core/src/test/java/org/apache/oozie/sla/TestSLACalculatorMemory.java 559e2b31 


Diff: https://reviews.apache.org/r/58406/diff/1/


Testing
---


Thanks,

Purshotam Shah

Review Request 58365: OOZIE-2862 Coord change command doesn't change job to running if job was killed without creating any actions

2017-04-11 Thread Purshotam Shah via Review Board 

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

Review request for oozie.


Bugs: OOZIE-2862
https://issues.apache.org/jira/browse/OOZIE-2862


Repository: oozie-git


Description
---

Coord change command doesn't change job to running if job was killed without 
creating any actions


Diffs
-

  core/src/main/java/org/apache/oozie/command/coord/CoordChangeXCommand.java 
e65b74fc 
  
core/src/test/java/org/apache/oozie/command/coord/TestCoordChangeXCommand.java 
8034bbe9 


Diff: https://reviews.apache.org/r/58365/diff/1/


Testing
---


Thanks,

Purshotam Shah

Re: Review Request 57107: OOZIE-2815 oozie not always display job log

2017-04-04 Thread Purshotam Shah via Review Board 

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

(Updated April 4, 2017, 7:41 p.m.)


Review request for oozie.


Bugs: OOZIE-2815
https://issues.apache.org/jira/browse/OOZIE-2815


Repository: oozie-git


Description
---

OOZIE-2815 oozie not always display job log


Diffs (updated)
-

  core/src/main/java/org/apache/oozie/BaseEngine.java 
50df8978773ed54fb5a39bb142d7235de1ed396f 
  core/src/main/java/org/apache/oozie/BundleEngine.java 
d0099b48772069217375c032d171e8dc6b1d 
  core/src/main/java/org/apache/oozie/CoordinatorEngine.java 
2f9f8227f1b2bbb0074cfa19aac37ab9a3fe7c0f 
  core/src/main/java/org/apache/oozie/DagEngine.java 
57d276199de149b294f64570a8878b660e9a5a0c 
  core/src/main/java/org/apache/oozie/service/XLogService.java 
04f04f41403fc6446967524124b8cf86f816d71c 
  core/src/main/java/org/apache/oozie/service/XLogStreamingService.java 
c15c4c1731838655d8f899f9c0405b226f8111b0 
  core/src/main/java/org/apache/oozie/service/ZKXLogStreamingService.java 
97771ad22be84d1a8167c69a6b48b7012652cf8b 
  core/src/main/java/org/apache/oozie/servlet/V0JobServlet.java 
d3b468965f33e300d7ca0f31efc596fd969b47a5 
  core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 
9356768639e6e3ea548b25baa75d576351fa80e3 
  core/src/main/java/org/apache/oozie/util/AuthUrlClient.java 
b45a96acc397c26cdba8d5c62503013dd572f247 
  core/src/main/java/org/apache/oozie/util/TimestampedMessageParser.java 
a676f4d35a49147aadde73357397b211ab829c85 
  core/src/main/java/org/apache/oozie/util/XLogAuditFilter.java 
c377db5df5dd0bbeed77b4d5c88e1c4e2e64a51e 
  core/src/main/java/org/apache/oozie/util/XLogAuditStreamer.java 
e69de29bb2d1d6434b8b29ae775ad8c2e48c5391 
  core/src/main/java/org/apache/oozie/util/XLogErrorStreamer.java 
e69de29bb2d1d6434b8b29ae775ad8c2e48c5391 
  core/src/main/java/org/apache/oozie/util/XLogFilter.java 
3b49f77e016cbeaedfa03708dbc680e68f40ca16 
  core/src/main/java/org/apache/oozie/util/XLogStreamer.java 
19f1fee55ba8d6c456e798f01e55cfe84cfb5803 
  core/src/main/resources/oozie-default.xml 
95e0c36e1b2f5aef18637f6f956a990ba36166f2 
  core/src/test/java/org/apache/oozie/TestCoordinatorEngineStreamLog.java 
3eb1016f3ffccaa9db077580b90cc1b3c53fd479 
  core/src/test/java/org/apache/oozie/service/TestConfigurationService.java 
42ffdbeb41e5829cdf814cf13ada331973fc33bd 
  core/src/test/java/org/apache/oozie/service/TestXLogStreamingService.java 
bebb678798ec4a01c9710660d4b99fddd5167560 
  core/src/test/java/org/apache/oozie/service/TestZKXLogStreamingService.java 
fca8d844f1563da5856eff36f53f21d519630d31 
  core/src/test/java/org/apache/oozie/util/TestLogStreamer.java 
f90f784503e101ae2f96fee52977ee83f3098301 
  
core/src/test/java/org/apache/oozie/util/TestSimplifiedTimestampedMessageParser.java
 ea899faf00aa2f98f43838987c9b716321a50dea 
  core/src/test/java/org/apache/oozie/util/TestTimestampedMessageParser.java 
9e28cbc77129bb2675b92ec9c18c77accfd78a55 
  core/src/test/java/org/apache/oozie/util/TestXLogUserFilterParam.java 
46f273fde7f14b446463448c90f22519761997b6 
  webapp/src/main/webapp/oozie-console.js 
76864a9a06e98357c2d17e35609e2a76afbd595c 


Diff: https://reviews.apache.org/r/57107/diff/3/

Changes: https://reviews.apache.org/r/57107/diff/2-3/


Testing
---


Thanks,

Purshotam Shah

Re: Review Request 57107: OOZIE-2815 oozie not always display job log

2017-04-04 Thread Purshotam Shah via Review Board 


> On March 31, 2017, 5:48 p.m., Satish Saley wrote:
> > core/src/main/java/org/apache/oozie/service/ZKXLogStreamingService.java
> > Line 324 (original), 251 (patched)
> > 
> >
> > I see that we are truncating logs for XLogErrorStreamer and 
> > XLogAuditStreamer rather than throwing exception.
> > But still there could be exeptions (may not be explicitly thrown) such 
> > as IOException from TimestampedMessageParser.increment() method. It would 
> > be safer to have a catch block, setting isException=true and flushing to 
> > the writer only if isException is false. That will take care of OOZIE-2636

We don't have to call flush on finally block. Writer is HttpServletResponse 
writer. In the end, the container will flush and close the HttpServletResponse 
stream.


> On March 31, 2017, 5:48 p.m., Satish Saley wrote:
> > core/src/main/java/org/apache/oozie/service/ZKXLogStreamingService.java
> > Line 324 (original), 251 (patched)
> > 
> >
> > I see that we are truncating logs for XLogErrorStreamer and 
> > XLogAuditStreamer rather than throwing exception.
> > But still there could be exeptions (may not be explicitly thrown) such 
> > as IOException from TimestampedMessageParser.increment() method. It would 
> > be safer to have a catch block, setting isException=true and flushing to 
> > the writer only if isException is false. That will take care of OOZIE-2636

We don't have to call flush on finally block. Writer is HttpServletResponse 
writer. In the end, the container will flush and close the HttpServletResponse 
stream.


- Purshotam


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


On March 14, 2017, 2:03 a.m., Purshotam Shah wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/57107/
> ---
> 
> (Updated March 14, 2017, 2:03 a.m.)
> 
> 
> Review request for oozie.
> 
> 
> Bugs: OOZIE-2815
> https://issues.apache.org/jira/browse/OOZIE-2815
> 
> 
> Repository: oozie-git
> 
> 
> Description
> ---
> 
> OOZIE-2815 oozie not always display job log
> 
> 
> Diffs
> -
> 
>   core/src/main/java/org/apache/oozie/BaseEngine.java 
> 50df8978773ed54fb5a39bb142d7235de1ed396f 
>   core/src/main/java/org/apache/oozie/BundleEngine.java 
> d0099b48772069217375c032d171e8dc6b1d 
>   core/src/main/java/org/apache/oozie/CoordinatorEngine.java 
> 2f9f8227f1b2bbb0074cfa19aac37ab9a3fe7c0f 
>   core/src/main/java/org/apache/oozie/DagEngine.java 
> 57d276199de149b294f64570a8878b660e9a5a0c 
>   core/src/main/java/org/apache/oozie/service/XLogService.java 
> 04f04f41403fc6446967524124b8cf86f816d71c 
>   core/src/main/java/org/apache/oozie/service/XLogStreamingService.java 
> c15c4c1731838655d8f899f9c0405b226f8111b0 
>   core/src/main/java/org/apache/oozie/service/ZKXLogStreamingService.java 
> 97771ad22be84d1a8167c69a6b48b7012652cf8b 
>   core/src/main/java/org/apache/oozie/servlet/V0JobServlet.java 
> d3b468965f33e300d7ca0f31efc596fd969b47a5 
>   core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 
> 9356768639e6e3ea548b25baa75d576351fa80e3 
>   core/src/main/java/org/apache/oozie/util/AuthUrlClient.java 
> b45a96acc397c26cdba8d5c62503013dd572f247 
>   core/src/main/java/org/apache/oozie/util/TimestampedMessageParser.java 
> a676f4d35a49147aadde73357397b211ab829c85 
>   core/src/main/java/org/apache/oozie/util/XLogAuditFilter.java 
> c377db5df5dd0bbeed77b4d5c88e1c4e2e64a51e 
>   core/src/main/java/org/apache/oozie/util/XLogAuditStreamer.java 
> e69de29bb2d1d6434b8b29ae775ad8c2e48c5391 
>   core/src/main/java/org/apache/oozie/util/XLogErrorStreamer.java 
> e69de29bb2d1d6434b8b29ae775ad8c2e48c5391 
>   core/src/main/java/org/apache/oozie/util/XLogFilter.java 
> 3b49f77e016cbeaedfa03708dbc680e68f40ca16 
>   core/src/main/java/org/apache/oozie/util/XLogStreamer.java 
> 19f1fee55ba8d6c456e798f01e55cfe84cfb5803 
>   core/src/main/resources/oozie-default.xml 
> 95e0c36e1b2f5aef18637f6f956a990ba36166f2 
>   core/src/test/java/org/apache/oozie/TestCoordinatorEngineStreamLog.java 
> 3eb1016f3ffccaa9db077580b90cc1b3c53fd479 
>   core/src/test/java/org/apache/oozie/service/TestConfigurationService.java 
> 42ffdbeb41e5829cdf814cf13ada331973fc33bd 
>   core/src/test/java/org/apache/oozie/service/TestXLogStreamingService.java 
> bebb678798ec4a01c9710660d4b99fddd5167560 
>   core/src/test/java/org/apache/oozie/service/TestZKXLogStreamingService.java 
> fca8d844f1563da5856eff36f53f21d519630d31 
>   core/src/test/java/org/apache/oozie/util/TestLogStreamer.java 
> f90f784503e101ae2f96fee52977ee83f3098301 
>   
>

Review Request 57107: OOZIE-2815 oozie not always display job log

2017-03-08 Thread Purshotam Shah via Review Board 

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

Review request for oozie.


Bugs: OOZIE-2815
https://issues.apache.org/jira/browse/OOZIE-2815


Repository: oozie-git


Description
---

OOZIE-2815 oozie not always display job log


Diffs
-

  core/src/main/java/org/apache/oozie/BaseEngine.java 
50df8978773ed54fb5a39bb142d7235de1ed396f 
  core/src/main/java/org/apache/oozie/BundleEngine.java 
d0099b48772069217375c032d171e8dc6b1d 
  core/src/main/java/org/apache/oozie/CoordinatorEngine.java 
2f9f8227f1b2bbb0074cfa19aac37ab9a3fe7c0f 
  core/src/main/java/org/apache/oozie/DagEngine.java 
57d276199de149b294f64570a8878b660e9a5a0c 
  core/src/main/java/org/apache/oozie/service/XLogService.java 
04f04f41403fc6446967524124b8cf86f816d71c 
  core/src/main/java/org/apache/oozie/service/XLogStreamingService.java 
c15c4c1731838655d8f899f9c0405b226f8111b0 
  core/src/main/java/org/apache/oozie/service/ZKXLogStreamingService.java 
97771ad22be84d1a8167c69a6b48b7012652cf8b 
  core/src/main/java/org/apache/oozie/servlet/V0JobServlet.java 
d3b468965f33e300d7ca0f31efc596fd969b47a5 
  core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java 
9356768639e6e3ea548b25baa75d576351fa80e3 
  core/src/main/java/org/apache/oozie/util/AuthUrlClient.java 
b45a96acc397c26cdba8d5c62503013dd572f247 
  core/src/main/java/org/apache/oozie/util/TimestampedMessageParser.java 
a676f4d35a49147aadde73357397b211ab829c85 
  core/src/main/java/org/apache/oozie/util/XLogAuditFilter.java 
c377db5df5dd0bbeed77b4d5c88e1c4e2e64a51e 
  core/src/main/java/org/apache/oozie/util/XLogAuditStreamer.java 
e69de29bb2d1d6434b8b29ae775ad8c2e48c5391 
  core/src/main/java/org/apache/oozie/util/XLogErrorStreamer.java 
e69de29bb2d1d6434b8b29ae775ad8c2e48c5391 
  core/src/main/java/org/apache/oozie/util/XLogFilter.java 
3b49f77e016cbeaedfa03708dbc680e68f40ca16 
  core/src/main/java/org/apache/oozie/util/XLogStreamer.java 
19f1fee55ba8d6c456e798f01e55cfe84cfb5803 
  core/src/main/resources/oozie-default.xml 
95e0c36e1b2f5aef18637f6f956a990ba36166f2 
  core/src/test/java/org/apache/oozie/TestCoordinatorEngineStreamLog.java 
3eb1016f3ffccaa9db077580b90cc1b3c53fd479 
  core/src/test/java/org/apache/oozie/service/TestConfigurationService.java 
42ffdbeb41e5829cdf814cf13ada331973fc33bd 
  core/src/test/java/org/apache/oozie/service/TestXLogStreamingService.java 
bebb678798ec4a01c9710660d4b99fddd5167560 
  core/src/test/java/org/apache/oozie/service/TestZKXLogStreamingService.java 
fca8d844f1563da5856eff36f53f21d519630d31 
  core/src/test/java/org/apache/oozie/util/TestLogStreamer.java 
f90f784503e101ae2f96fee52977ee83f3098301 
  
core/src/test/java/org/apache/oozie/util/TestSimplifiedTimestampedMessageParser.java
 ea899faf00aa2f98f43838987c9b716321a50dea 
  core/src/test/java/org/apache/oozie/util/TestTimestampedMessageParser.java 
9e28cbc77129bb2675b92ec9c18c77accfd78a55 
  core/src/test/java/org/apache/oozie/util/TestXLogUserFilterParam.java 
46f273fde7f14b446463448c90f22519761997b6 
  webapp/src/main/webapp/oozie-console.js 
76864a9a06e98357c2d17e35609e2a76afbd595c 


Diff: https://reviews.apache.org/r/57107/diff/1/


Testing
---


Thanks,

Purshotam Shah