From 1c212bf79a90a42a3a0297482ed118c27e66dceb Mon Sep 17 00:00:00 2001 From: Ian Macalinao Date: Sun, 12 Mar 2017 18:02:43 -0500 Subject: [PATCH] Remove use of optionals in union; declare TProgressUpdateResp before its usage --- service-rpc/if/TCLIService.thrift | 34 +++++++++++++++++----------------- 1 file changed, 17 insertions(+), 17 deletions(-) diff --git a/service-rpc/if/TCLIService.thrift b/service-rpc/if/TCLIService.thrift index 824b049..b886260 100644 --- a/service-rpc/if/TCLIService.thrift +++ b/service-rpc/if/TCLIService.thrift @@ -203,8 +203,8 @@ const string PRECISION = "precision" const string SCALE = "scale" union TTypeQualifierValue { - 1: optional i32 i32Value - 2: optional string stringValue + 1: i32 i32Value + 2: string stringValue } // Type qualifiers for primitive type. @@ -1016,6 +1016,21 @@ struct TGetCrossReferenceResp { 2: optional TOperationHandle operationHandle } +enum TJobExecutionStatus { + IN_PROGRESS, + COMPLETE, + NOT_AVAILABLE +} + +struct TProgressUpdateResp { + 1: required list headerNames + 2: required list> rows + 3: required double progressedPercentage + 4: required TJobExecutionStatus status + 5: required string footerSummary + 6: required i64 startTime +} + // GetOperationStatus() // // Get the status of an operation running on the server. @@ -1209,21 +1224,6 @@ struct TRenewDelegationTokenResp { 1: required TStatus status } -enum TJobExecutionStatus { - IN_PROGRESS, - COMPLETE, - NOT_AVAILABLE -} - -struct TProgressUpdateResp { - 1: required list headerNames - 2: required list> rows - 3: required double progressedPercentage - 4: required TJobExecutionStatus status - 5: required string footerSummary - 6: required i64 startTime -} - service TCLIService { TOpenSessionResp OpenSession(1:TOpenSessionReq req); -- 2.10.1 (Apple Git-78)