[TAJO-1356] Race conditions in QueryInProgress

• Type: Bug
• Status: Resolved
• Priority: Critical
• Resolution: Fixed
• Affects Version/s: 0.10.0, 0.11.0
• Fix Version/s: 0.10.0, 0.11.0
• Component/s: TajoMaster
• Assignee: Jinho Kim
• Reporter: Jinho Kim
• Created: 23/Feb/15 08:53
• Updated: 23/Feb/15 11:37
• Resolved: 23/Feb/15 11:21

Description

TajoMasterClientService.getQueryResultData is intermittently failing due to a race condition. This issue can be fixed by removing unnecessary async condition and adding correctly synchronization in QueryInProgress.

-------------------------------------------------------------------------------
Test set: org.apache.tajo.engine.query.TestSortQuery
-------------------------------------------------------------------------------
Tests run: 23, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 40.597 sec <<< FAILURE! - in org.apache.tajo.engine.query.TestSortQuery
testSortWithAlias3(org.apache.tajo.engine.query.TestSortQuery)  Time elapsed: 2.12 sec  <<< ERROR!
java.sql.SQLException: com.google.protobuf.ServiceException: QueryInfo cannot be NULL.
        at org.apache.tajo.client.QueryClientImpl$3.call(QueryClientImpl.java:393)
        at org.apache.tajo.client.QueryClientImpl$3.call(QueryClientImpl.java:378)
        at org.apache.tajo.rpc.ServerCallable.withRetries(ServerCallable.java:94)
        at org.apache.tajo.client.QueryClientImpl.fetchNextQueryResult(QueryClientImpl.java:406)
        at org.apache.tajo.jdbc.FetchResultSet.nextTuple(FetchResultSet.java:66)
        at org.apache.tajo.jdbc.TajoResultSetBase.next(TajoResultSetBase.java:780)
        at org.apache.tajo.QueryTestCaseBase.resultSetToString(QueryTestCaseBase.java:497)
        at org.apache.tajo.engine.query.TestSortQuery.testSortWithAlias3(TestSortQuery.java:71)