@celerispay/hazelcast-client 3.12.5-6 → 3.12.5-8

package/lib/PartitionService.js

@@ -69,11 +69,13 @@ var PartitionService = /** @class */ (function () {
         var ownerConnection = this.client.getClusterService().getOwnerConnection();
         if (ownerConnection == null) {
             this.logger.warn('PartitionService', 'Cannot refresh partitions, no owner connection available');
-            return Promise.resolve();
+            // Return a rejected promise instead of resolved to indicate failure
+            return Promise.reject(new Error('No owner connection available for partition refresh'));
         }
         this.refreshInProgress = true;
         var clientMessage = GetPartitionsCodec.encodeRequest();
-        return this.client.getInvocationService()
+        // Add timeout to prevent hanging
+        var refreshPromise = this.client.getInvocationService()
             .invokeOnConnection(ownerConnection, clientMessage)
             .then(function (response) {
             var receivedPartitionMap = GetPartitionsCodec.decodeResponse(response);
@@ -93,9 +95,17 @@ var PartitionService = /** @class */ (function () {
                     _this.clearPartitionTable();
                 }
             }
+            throw e; // Re-throw the error to propagate it
         }).finally(function () {
             _this.refreshInProgress = false;
         });
+        // Add timeout to prevent hanging
+        var timeoutPromise = new Promise(function (_, reject) {
+            setTimeout(function () {
+                reject(new Error('Partition refresh timed out'));
+            }, 10000); // 10 second timeout
+        });
+        return Promise.race([refreshPromise, timeoutPromise]);
     };
     /**
      * Returns the {@link Address} of the node which owns given partition id.

package/lib/invocation/InvocationService.js

@@ -289,6 +289,10 @@ var InvocationService = /** @class */ (function () {
                     throw new Error("Still no partition owner for partition " + partitionId + " after refresh");
                 }
                 return _this.invokeOnAddress(invocation, newOwnerAddress);
+            }).catch(function (error) {
+                _this.logger.error('InvocationService', "Failed to refresh partition table for partition " + partitionId + ":", error);
+                // If partition refresh fails, reject the invocation instead of hanging
+                throw new Error("Cannot find partition owner for partition " + partitionId + ": " + error.message);
             });
         }
         return this.client.getConnectionManager().getOrConnect(ownerAddress).then(function (connection) {
@@ -301,6 +305,10 @@ var InvocationService = /** @class */ (function () {
                 return _this.client.getPartitionService().refresh().then(function () {
                     // Retry the invocation with updated partition information
                     return _this.doInvoke(invocation);
+                }).catch(function (refreshError) {
+                    _this.logger.error('InvocationService', "Failed to refresh partition table after partition owner failure:", refreshError);
+                    // If refresh fails, reject the invocation instead of hanging
+                    throw new Error("Partition owner " + ownerAddress.toString() + " unavailable and partition refresh failed: " + refreshError.message);
                 });
             }
             throw new HazelcastError_1.IOError(ownerAddress.toString() + '(partition owner) is not available.', e);

package/lib/proxy/ProxyManager.d.ts

@@ -37,8 +37,4 @@ export declare class ProxyManager {
     private findNextAddress();
     private initializeProxy(proxyObject, promise, deadline);
     private createDistributedObjectListener();
-    /**
-     * Checks if the cluster is healthy enough to create proxies
-     */
-    private isClusterHealthy();
 }

package/lib/proxy/ProxyManager.js

@@ -15,7 +15,6 @@
  * limitations under the License.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-var Promise = require("bluebird");
 var ClientAddDistributedObjectListenerCodec_1 = require("../codec/ClientAddDistributedObjectListenerCodec");
 var ClientCreateProxyCodec_1 = require("../codec/ClientCreateProxyCodec");
 var ClientDestroyProxyCodec_1 = require("../codec/ClientDestroyProxyCodec");
@@ -69,12 +68,6 @@ var ProxyManager = /** @class */ (function () {
         if (this.proxies[fullName]) {
             return this.proxies[fullName];
         }
-        // Check if cluster is healthy before creating proxy
-        if (!this.isClusterHealthy()) {
-            var error = new Error('Cluster is not healthy, cannot create proxy for ' + name);
-            this.logger.error('ProxyManager', error.message);
-            return Promise.reject(error);
-        }
         var deferred = Util_1.DeferredPromise();
         var newProxy;
         if (serviceName === ProxyManager.MAP_SERVICE && this.client.getConfig().getNearCacheConfig(name)) {
@@ -98,9 +91,6 @@ var ProxyManager = /** @class */ (function () {
         if (createAtServer) {
             this.createProxy(newProxy).then(function () {
                 deferred.resolve(newProxy);
-            }).catch(function (error) {
-                _this.logger.error('ProxyManager', 'Failed to create proxy for ' + name + ': ' + error);
-                deferred.reject(error);
             });
         }
         this.proxies[fullName] = deferred.promise;
@@ -171,37 +161,36 @@ var ProxyManager = /** @class */ (function () {
     };
     ProxyManager.prototype.initializeProxy = function (proxyObject, promise, deadline) {
         var _this = this;
-        if (Date.now() > deadline) {
-            var error = new Error('Create proxy request timed-out for ' + proxyObject.getName());
-            this.logger.error('ProxyManager', error.message);
-            promise.reject(error);
-            return;
-        }
-        var address = this.findNextAddress();
-        if (!address) {
-            var error = new Error('No cluster members available for proxy creation: ' + proxyObject.getName());
-            this.logger.error('ProxyManager', error.message);
-            promise.reject(error);
-            return;
-        }
-        var request = ClientCreateProxyCodec_1.ClientCreateProxyCodec.encodeRequest(proxyObject.getName(), proxyObject.getServiceName(), address);
-        var invocation = new InvocationService_1.Invocation(this.client, request);
-        invocation.address = address;
-        this.client.getInvocationService().invoke(invocation).then(function (response) {
-            promise.resolve(response);
-        }).catch(function (error) {
-            if (_this.isRetryable(error)) {
-                _this.logger.warn('ProxyManager', 'Create proxy request for ' + proxyObject.getName() +
-                    ' failed. Retrying in ' + _this.invocationRetryPauseMillis + 'ms. ' + error);
-                setTimeout(function () {
-                    _this.initializeProxy(proxyObject, promise, deadline);
-                }, _this.invocationRetryPauseMillis);
-            }
-            else {
-                _this.logger.error('ProxyManager', 'Create proxy request for ' + proxyObject.getName() + ' failed ' + error);
+        if (Date.now() <= deadline) {
+            var address = this.findNextAddress();
+            if (!address) {
+                var error = new Error('No cluster members available for proxy creation: ' + proxyObject.getName());
+                this.logger.error('ProxyManager', error.message);
                 promise.reject(error);
+                return;
             }
-        });
+            var request = ClientCreateProxyCodec_1.ClientCreateProxyCodec.encodeRequest(proxyObject.getName(), proxyObject.getServiceName(), address);
+            var invocation = new InvocationService_1.Invocation(this.client, request);
+            invocation.address = address;
+            this.client.getInvocationService().invoke(invocation).then(function (response) {
+                promise.resolve(response);
+            }).catch(function (error) {
+                if (_this.isRetryable(error)) {
+                    _this.logger.warn('ProxyManager', 'Create proxy request for ' + proxyObject.getName() +
+                        ' failed. Retrying in ' + _this.invocationRetryPauseMillis + 'ms. ' + error);
+                    setTimeout(function () {
+                        _this.initializeProxy(proxyObject, promise, deadline);
+                    }, _this.invocationRetryPauseMillis);
+                }
+                else {
+                    _this.logger.error('ProxyManager', 'Create proxy request for ' + proxyObject.getName() + ' failed ' + error);
+                    promise.reject(error);
+                }
+            });
+        }
+        else {
+            promise.reject('Create proxy request timed-out for ' + proxyObject.getName());
+        }
     };
     ProxyManager.prototype.createDistributedObjectListener = function () {
         return {
@@ -216,30 +205,6 @@ var ProxyManager = /** @class */ (function () {
             },
         };
     };
-    /**
-     * Checks if the cluster is healthy enough to create proxies
-     */
-    ProxyManager.prototype.isClusterHealthy = function () {
-        var members = this.client.getClusterService().getMembers();
-        var hasMembers = members && members.length > 0;
-        if (!hasMembers) {
-            this.logger.warn('ProxyManager', 'No cluster members available');
-            return false;
-        }
-        // Check if we have at least one data member
-        var hasDataMember = members.some(function (member) { return member && !member.isLiteMember; });
-        if (!hasDataMember) {
-            this.logger.warn('ProxyManager', 'No data members available in cluster');
-            return false;
-        }
-        // Check if we have an owner connection
-        var ownerConnection = this.client.getClusterService().getOwnerConnection();
-        if (!ownerConnection || !ownerConnection.isHealthy()) {
-            this.logger.warn('ProxyManager', 'No healthy owner connection available');
-            return false;
-        }
-        return true;
-    };
     ProxyManager.MAP_SERVICE = 'hz:impl:mapService';
     ProxyManager.SET_SERVICE = 'hz:impl:setService';
     ProxyManager.LOCK_SERVICE = 'hz:impl:lockService';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@celerispay/hazelcast-client",
-  "version": "3.12.5-6",
+  "version": "3.12.5-8",
   "description": "Hazelcast - open source In-Memory Data Grid - client for NodeJS with critical connection failover fixes",
   "main": "./lib/index.js",
   "scripts": {