Commit 41391dbef8bbfbf1c0f56165dd41277dc45d23fe

Authored by Vladyslav_Prykhodko
1 parent 1d9fc4a3

js-executor: code format

@@ -43,7 +43,7 @@ function JsInvokeMessageProcessor(producer) { @@ -43,7 +43,7 @@ function JsInvokeMessageProcessor(producer) {
43 this.lastStatTime = performance.now(); 43 this.lastStatTime = performance.now();
44 } 44 }
45 45
46 -JsInvokeMessageProcessor.prototype.onJsInvokeMessage = function(message) { 46 +JsInvokeMessageProcessor.prototype.onJsInvokeMessage = function (message) {
47 var tStart = performance.now(); 47 var tStart = performance.now();
48 let requestId; 48 let requestId;
49 let responseTopic; 49 let responseTopic;
@@ -78,13 +78,13 @@ JsInvokeMessageProcessor.prototype.onJsInvokeMessage = function(message) { @@ -78,13 +78,13 @@ JsInvokeMessageProcessor.prototype.onJsInvokeMessage = function(message) {
78 var tFinish = performance.now(); 78 var tFinish = performance.now();
79 var tTook = tFinish - tStart; 79 var tTook = tFinish - tStart;
80 80
81 - if ( tTook > slowQueryLogMs ) { 81 + if (tTook > slowQueryLogMs) {
82 let functionName; 82 let functionName;
83 if (request.invokeRequest) { 83 if (request.invokeRequest) {
84 try { 84 try {
85 buf = Buffer.from(request.invokeRequest['functionName']); 85 buf = Buffer.from(request.invokeRequest['functionName']);
86 functionName = buf.toString('utf8'); 86 functionName = buf.toString('utf8');
87 - } catch (err){ 87 + } catch (err) {
88 logger.error('[%s] Failed to read functionName from message header: %s', requestId, err.message); 88 logger.error('[%s] Failed to read functionName from message header: %s', requestId, err.message);
89 logger.error(err.stack); 89 logger.error(err.stack);
90 } 90 }
@@ -97,7 +97,7 @@ JsInvokeMessageProcessor.prototype.onJsInvokeMessage = function(message) { @@ -97,7 +97,7 @@ JsInvokeMessageProcessor.prototype.onJsInvokeMessage = function(message) {
97 97
98 } 98 }
99 99
100 -JsInvokeMessageProcessor.prototype.processCompileRequest = function(requestId, responseTopic, headers, compileRequest) { 100 +JsInvokeMessageProcessor.prototype.processCompileRequest = function (requestId, responseTopic, headers, compileRequest) {
101 var scriptId = getScriptId(compileRequest); 101 var scriptId = getScriptId(compileRequest);
102 logger.debug('[%s] Processing compile request, scriptId: [%s]', requestId, scriptId); 102 logger.debug('[%s] Processing compile request, scriptId: [%s]', requestId, scriptId);
103 103
@@ -116,7 +116,7 @@ JsInvokeMessageProcessor.prototype.processCompileRequest = function(requestId, r @@ -116,7 +116,7 @@ JsInvokeMessageProcessor.prototype.processCompileRequest = function(requestId, r
116 ); 116 );
117 } 117 }
118 118
119 -JsInvokeMessageProcessor.prototype.processInvokeRequest = function(requestId, responseTopic, headers, invokeRequest) { 119 +JsInvokeMessageProcessor.prototype.processInvokeRequest = function (requestId, responseTopic, headers, invokeRequest) {
120 var scriptId = getScriptId(invokeRequest); 120 var scriptId = getScriptId(invokeRequest);
121 logger.debug('[%s] Processing invoke request, scriptId: [%s]', requestId, scriptId); 121 logger.debug('[%s] Processing invoke request, scriptId: [%s]', requestId, scriptId);
122 this.executedScriptsCounter++; 122 this.executedScriptsCounter++;
@@ -160,7 +160,7 @@ JsInvokeMessageProcessor.prototype.processInvokeRequest = function(requestId, re @@ -160,7 +160,7 @@ JsInvokeMessageProcessor.prototype.processInvokeRequest = function(requestId, re
160 ); 160 );
161 } 161 }
162 162
163 -JsInvokeMessageProcessor.prototype.processReleaseRequest = function(requestId, responseTopic, headers, releaseRequest) { 163 +JsInvokeMessageProcessor.prototype.processReleaseRequest = function (requestId, responseTopic, headers, releaseRequest) {
164 var scriptId = getScriptId(releaseRequest); 164 var scriptId = getScriptId(releaseRequest);
165 logger.debug('[%s] Processing release request, scriptId: [%s]', requestId, scriptId); 165 logger.debug('[%s] Processing release request, scriptId: [%s]', requestId, scriptId);
166 if (this.scriptMap.has(scriptId)) { 166 if (this.scriptMap.has(scriptId)) {
@@ -193,9 +193,9 @@ JsInvokeMessageProcessor.prototype.sendResponse = function (requestId, responseT @@ -193,9 +193,9 @@ JsInvokeMessageProcessor.prototype.sendResponse = function (requestId, responseT
193 ); 193 );
194 } 194 }
195 195
196 -JsInvokeMessageProcessor.prototype.getOrCompileScript = function(scriptId, scriptBody) { 196 +JsInvokeMessageProcessor.prototype.getOrCompileScript = function (scriptId, scriptBody) {
197 var self = this; 197 var self = this;
198 - return new Promise(function(resolve, reject) { 198 + return new Promise(function (resolve, reject) {
199 if (self.scriptMap.has(scriptId)) { 199 if (self.scriptMap.has(scriptId)) {
200 resolve(self.scriptMap.get(scriptId)); 200 resolve(self.scriptMap.get(scriptId));
201 } else { 201 } else {
@@ -212,7 +212,7 @@ JsInvokeMessageProcessor.prototype.getOrCompileScript = function(scriptId, scrip @@ -212,7 +212,7 @@ JsInvokeMessageProcessor.prototype.getOrCompileScript = function(scriptId, scrip
212 }); 212 });
213 } 213 }
214 214
215 -JsInvokeMessageProcessor.prototype.cacheScript = function(scriptId, script) { 215 +JsInvokeMessageProcessor.prototype.cacheScript = function (scriptId, script) {
216 if (!this.scriptMap.has(scriptId)) { 216 if (!this.scriptMap.has(scriptId)) {
217 this.scriptIds.push(scriptId); 217 this.scriptIds.push(scriptId);
218 while (this.scriptIds.length > maxActiveScripts) { 218 while (this.scriptIds.length > maxActiveScripts) {
@@ -229,40 +229,40 @@ JsInvokeMessageProcessor.prototype.cacheScript = function(scriptId, script) { @@ -229,40 +229,40 @@ JsInvokeMessageProcessor.prototype.cacheScript = function(scriptId, script) {
229 function createRemoteResponse(requestId, compileResponse, invokeResponse, releaseResponse) { 229 function createRemoteResponse(requestId, compileResponse, invokeResponse, releaseResponse) {
230 const requestIdBits = Utils.UUIDToBits(requestId); 230 const requestIdBits = Utils.UUIDToBits(requestId);
231 return { 231 return {
232 - requestIdMSB: requestIdBits[0],  
233 - requestIdLSB: requestIdBits[1],  
234 - compileResponse: compileResponse,  
235 - invokeResponse: invokeResponse,  
236 - releaseResponse: releaseResponse 232 + requestIdMSB: requestIdBits[0],
  233 + requestIdLSB: requestIdBits[1],
  234 + compileResponse: compileResponse,
  235 + invokeResponse: invokeResponse,
  236 + releaseResponse: releaseResponse
237 }; 237 };
238 } 238 }
239 239
240 function createCompileResponse(scriptId, success, errorCode, err) { 240 function createCompileResponse(scriptId, success, errorCode, err) {
241 const scriptIdBits = Utils.UUIDToBits(scriptId); 241 const scriptIdBits = Utils.UUIDToBits(scriptId);
242 - return {  
243 - errorCode: errorCode,  
244 - success: success,  
245 - errorDetails: parseJsErrorDetails(err),  
246 - scriptIdMSB: scriptIdBits[0],  
247 - scriptIdLSB: scriptIdBits[1] 242 + return {
  243 + errorCode: errorCode,
  244 + success: success,
  245 + errorDetails: parseJsErrorDetails(err),
  246 + scriptIdMSB: scriptIdBits[0],
  247 + scriptIdLSB: scriptIdBits[1]
248 }; 248 };
249 } 249 }
250 250
251 function createInvokeResponse(result, success, errorCode, err) { 251 function createInvokeResponse(result, success, errorCode, err) {
252 - return {  
253 - errorCode: errorCode,  
254 - success: success,  
255 - errorDetails: parseJsErrorDetails(err),  
256 - result: result 252 + return {
  253 + errorCode: errorCode,
  254 + success: success,
  255 + errorDetails: parseJsErrorDetails(err),
  256 + result: result
257 }; 257 };
258 } 258 }
259 259
260 function createReleaseResponse(scriptId, success) { 260 function createReleaseResponse(scriptId, success) {
261 const scriptIdBits = Utils.UUIDToBits(scriptId); 261 const scriptIdBits = Utils.UUIDToBits(scriptId);
262 return { 262 return {
263 - success: success,  
264 - scriptIdMSB: scriptIdBits[0],  
265 - scriptIdLSB: scriptIdBits[1] 263 + success: success,
  264 + scriptIdMSB: scriptIdBits[0],
  265 + scriptIdLSB: scriptIdBits[1]
266 }; 266 };
267 } 267 }
268 268
@@ -45,9 +45,9 @@ function KafkaProducer() { @@ -45,9 +45,9 @@ function KafkaProducer() {
45 const message = { 45 const message = {
46 topic: responseTopic, 46 topic: responseTopic,
47 messages: [{ 47 messages: [{
48 - key: scriptId,  
49 - value: rawResponse,  
50 - headers: headers.data 48 + key: scriptId,
  49 + value: rawResponse,
  50 + headers: headers.data
51 }] 51 }]
52 }; 52 };
53 53
@@ -70,28 +70,28 @@ function pushMessageToSendLater(message) { @@ -70,28 +70,28 @@ function pushMessageToSendLater(message) {
70 } 70 }
71 71
72 function sendLoopWithLinger() { 72 function sendLoopWithLinger() {
73 - if (sendLoopInstance) { 73 + if (sendLoopInstance) {
74 logger.debug("Clear sendLoop scheduler. Starting new send loop with linger [%s]", linger); 74 logger.debug("Clear sendLoop scheduler. Starting new send loop with linger [%s]", linger);
75 clearInterval(sendLoopInstance); 75 clearInterval(sendLoopInstance);
76 - } else { 76 + } else {
77 logger.debug("Starting new send loop with linger [%s]", linger) 77 logger.debug("Starting new send loop with linger [%s]", linger)
78 - }  
79 - sendLoopInstance = setInterval(sendMessagesAsBatch, linger); 78 + }
  79 + sendLoopInstance = setInterval(sendMessagesAsBatch, linger);
80 } 80 }
81 81
82 function sendMessagesAsBatch() { 82 function sendMessagesAsBatch() {
83 - if (batchMessages.length > 0) {  
84 - logger.debug('sendMessagesAsBatch, length: [%s]', batchMessages.length);  
85 - const messagesToSend = batchMessages;  
86 - const resolvers = batchResolvers;  
87 - batchMessages = [];  
88 - batchResolvers = [];  
89 - producer.sendBatch({  
90 - topicMessages: messagesToSend,  
91 - acks: acks,  
92 - compression: compressionType  
93 - }).then(  
94 - () => { 83 + if (batchMessages.length > 0) {
  84 + logger.debug('sendMessagesAsBatch, length: [%s]', batchMessages.length);
  85 + const messagesToSend = batchMessages;
  86 + const resolvers = batchResolvers;
  87 + batchMessages = [];
  88 + batchResolvers = [];
  89 + producer.sendBatch({
  90 + topicMessages: messagesToSend,
  91 + acks: acks,
  92 + compression: compressionType
  93 + }).then(
  94 + () => {
95 logger.debug('Response batch sent to kafka, length: [%s]', messagesToSend.length); 95 logger.debug('Response batch sent to kafka, length: [%s]', messagesToSend.length);
96 for (let i = 0; i < resolvers.length; i++) { 96 for (let i = 0; i < resolvers.length; i++) {
97 resolvers[i](); 97 resolvers[i]();
@@ -103,8 +103,8 @@ function sendMessagesAsBatch() { @@ -103,8 +103,8 @@ function sendMessagesAsBatch() {
103 batchMessages = messagesToSend.concat(batchMessages); 103 batchMessages = messagesToSend.concat(batchMessages);
104 batchResolvers = resolvers.concat(batchResolvers); //promises will never be rejected. Will retry forever 104 batchResolvers = resolvers.concat(batchResolvers); //promises will never be rejected. Will retry forever
105 } 105 }
106 - );  
107 - } 106 + );
  107 + }
108 } 108 }
109 109
110 (async () => { 110 (async () => {
@@ -120,8 +120,8 @@ function sendMessagesAsBatch() { @@ -120,8 +120,8 @@ function sendMessagesAsBatch() {
120 120
121 let kafkaConfig = { 121 let kafkaConfig = {
122 brokers: kafkaBootstrapServers.split(','), 122 brokers: kafkaBootstrapServers.split(','),
123 - logLevel: logLevel.INFO,  
124 - logCreator: KafkaJsWinstonLogCreator 123 + logLevel: logLevel.INFO,
  124 + logCreator: KafkaJsWinstonLogCreator
125 }; 125 };
126 126
127 if (kafkaClientId) { 127 if (kafkaClientId) {