[ https://issues.apache.org/jira/browse/OOZIE-3125?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16258579#comment-16258579 ]
Hadoop QA commented on OOZIE-3125: ---------------------------------- Testing JIRA OOZIE-3125 Cleaning local git workspace ---------------------------- {color:green}+1 PATCH_APPLIES{color} {color:green}+1 CLEAN{color} {color:red}-1 RAW_PATCH_ANALYSIS{color} . {color:green}+1{color} the patch does not introduce any @author tags . {color:green}+1{color} the patch does not introduce any tabs . {color:green}+1{color} the patch does not introduce any trailing spaces . {color:red}-1{color} the patch contains 4 line(s) longer than 132 characters . {color:green}+1{color} the patch does adds/modifies 4 testcase(s) {color:green}+1 RAT{color} . {color:green}+1{color} the patch does not seem to introduce new RAT warnings {color:green}+1 JAVADOC{color} . {color:green}+1{color} the patch does not seem to introduce new Javadoc warnings . {color:red}WARNING{color}: the current HEAD has 77 Javadoc warning(s) {color:green}+1 COMPILE{color} . {color:green}+1{color} HEAD compiles . {color:green}+1{color} patch compiles . {color:green}+1{color} the patch does not seem to introduce new javac warnings {color:green}+1{color} There are no new bugs found in total. . {color:green}+1{color} There are no new bugs found in [docs]. . {color:green}+1{color} There are no new bugs found in [sharelib/distcp]. . {color:green}+1{color} There are no new bugs found in [sharelib/hive]. . {color:green}+1{color} There are no new bugs found in [sharelib/spark]. . {color:green}+1{color} There are no new bugs found in [sharelib/hive2]. . {color:green}+1{color} There are no new bugs found in [sharelib/hcatalog]. . {color:green}+1{color} There are no new bugs found in [sharelib/streaming]. . {color:green}+1{color} There are no new bugs found in [sharelib/pig]. . {color:green}+1{color} There are no new bugs found in [sharelib/sqoop]. . {color:green}+1{color} There are no new bugs found in [sharelib/oozie]. . {color:green}+1{color} There are no new bugs found in [examples]. . {color:green}+1{color} There are no new bugs found in [client]. . {color:green}+1{color} There are no new bugs found in [core]. . {color:green}+1{color} There are no new bugs found in [tools]. . {color:green}+1{color} There are no new bugs found in [server]. {color:green}+1 BACKWARDS_COMPATIBILITY{color} . {color:green}+1{color} the patch does not change any JPA Entity/Colum/Basic/Lob/Transient annotations . {color:green}+1{color} the patch does not modify JPA files {color:red}-1 TESTS{color} . Tests run: 2069 . Tests failed: 1 . Tests errors: 0 . The patch failed the following testcases: . testJMXInstrumentation(org.apache.oozie.util.TestMetricsInstrumentation) . Tests failing with errors: . {color:green}+1 DISTRO{color} . {color:green}+1{color} distro tarball builds with the patch ---------------------------- {color:red}*-1 Overall result, please check the reported -1(s)*{color} {color:red}. There is at least one warning, please check{color} The full output of the test-patch run is available at . https://builds.apache.org/job/PreCommit-OOZIE-Build/243/ > TestDBLoadDump.testImportInvalidDataLeavesTablesEmpty fails > ----------------------------------------------------------- > > Key: OOZIE-3125 > URL: https://issues.apache.org/jira/browse/OOZIE-3125 > Project: Oozie > Issue Type: Bug > Components: tests > Reporter: Attila Sasvari > Assignee: Attila Sasvari > Priority: Minor > Fix For: 5.0.0b1 > > Attachments: OOZIE-3125-001.patch, OOZIE-3125-002.patch, > OOZIE-3125-amend-1.patch, OOZIE-3125-amend-2.patch > > > Running {{mvn test > -Dtest=TestDBLoadDump#testImportInvalidDataLeavesTablesEmpty}} results in a > test failure: > {code} > [ERROR] > TestDBLoadDump.testImportInvalidDataLeavesTablesEmpty:136->tryImportAndCheckPrematureExit:271 > import should have been ended prematurely > {code} > {{TestDBLoadDump.testImportInvalidDataLeavesTablesEmpty}} tests that > processing an invalid dump file results in transactions are rolled back and > tables are left unpopulated (empty). > {{tools/src/test/resources/dumpData/invalid/ooziedb_ac.json}} contains an > {{executionPath}} that consists of more than 1024 characters (2000). > {{OozieDBImportCLI.importOneInputFileToOneEntityTable}} deserialize the > WorkflowActionBean object from the JSON using {{final E newEntity = > gson.fromJson(line, entityClass);}} . However, persisting this object should > fail even because invariants (see {{@Column(name = "execution_path", length = > 1024)}} in {{WorkflowActionBean}}) do not hold. > In the setup method of TestDBLoadDump, {{setSqlStricEnforce()}} is used to > make sure strict size is enforced: > {code} > private void setSqlStrictEnforce(final EntityManager entityManager) { > final String sqlStrictEnforce = "SET PROPERTY > \"sql.enforce_strict_size\" TRUE"; > final EntityTransaction tx = entityManager.getTransaction(); > tx.begin(); > entityManager.createNativeQuery(sqlStrictEnforce).executeUpdate(); > tx.commit(); > } > {code} > However it does not seem to take effect. [HSQL db's > documentation|http://hsqldb.org/doc/guide/dbproperties-chapt.html] says: > {quote} > Management of properties has changed since version 1.8. The old SET PROPERTY > statement does not change a property and is ignored. The statement is > retained to simplify application upgrades. > {quote} > Running {{mvn dependency:tree}}, it turns out Oozie core uses 1.8.0.10 > {code} > [INFO] +- hsqldb:hsqldb:jar:1.8.0.10:compile > {code} > It is, however, strange that tests pass if running all the test cases in > TestDBLoadDump with {{mvn test -Dmaven.surefire.debug > -Dtest=TestDBLoadDump}}. In other words, tests are not independent of > execution order. > Looking at a successfull test execution in surefire-reports > ({{TEST-org.apache.oozie.tools.TestDBLoadDump.xml}}) : > {code} > <testcase name="testImportTablesOverflowBatchSize" > classname="org.apache.oozie.tools.TestDBLoadDump" time="25.362"/> > <testcase name="testImportToNonExistingTablesSucceedsOnHsqldb" > classname="org.apache.oozie.tools.TestDBLoadDump" time="0.559"/> > <testcase name="testImportInvalidDataLeavesTablesEmpty" > classname="org.apache.oozie.tools.TestDBLoadDump" time="0.708"/> > <testcase name="testImportToNonEmptyTablesCausesPrematureExit" > classname="org.apache.oozie.tools.TestDBLoadDump" time="1.066"/> > <testcase name="testImportedDBIsExportedCorrectly" > classname="org.apache.oozie.tools.TestDBLoadDump" time="1.135"/> > {code} > We shall make sure individual tests pass. As a first step, > {{TestDBLoadDump.testImportInvalidDataLeavesTablesEmpty}} is to be fixed. -- This message was sent by Atlassian JIRA (v6.4.14#64029)