@cap-js-community/event-queue 1.9.0-beta.1 → 1.9.0-beta.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cap-js-community/event-queue",
-  "version": "1.9.0-beta.1",
+  "version": "1.9.0-beta.2",
   "description": "An event queue that enables secure transactional processing of asynchronous and periodic events, featuring instant event processing with Redis Pub/Sub and load distribution across all application instances.",
   "main": "src/index.js",
   "types": "src/index.d.ts",

package/src/EventQueueProcessorBase.js

@@ -852,7 +852,7 @@ class EventQueueProcessorBase {
   // eslint-disable-next-line no-unused-vars
   async beforeProcessingEvents() {}
 
-  async isOutdatedAndKeepalive() {
+  async isOutdatedAndKeepAlive() {
     if (this.__keepAliveViolated) {
       return true;
     }

package/src/processEventQueue.js

@@ -159,6 +159,7 @@ const processPeriodicEvent = async (context, eventTypeInstance) => {
         `eventQueue-periodic-process-${eventTypeInstance.eventType}##${eventTypeInstance.eventSubType}`,
         async (tx) => {
           await trace(eventTypeInstance.context, "process-periodic-event", async () => {
+            eventTypeInstance.continuesKeepAlive();
             eventTypeInstance.processEventContext = tx.context;
             eventTypeInstance.setTxForEventProcessing(queueEntry.ID, cds.tx(tx.context));
             try {
@@ -170,6 +171,7 @@ const processPeriodicEvent = async (context, eventTypeInstance) => {
               await tx.rollback();
               return;
             } finally {
+              eventTypeInstance.stopKeepAlive();
               eventTypeInstance.endPerformanceTracerPeriodicEvents();
             }
             if (
@@ -198,6 +200,8 @@ const processPeriodicEvent = async (context, eventTypeInstance) => {
       eventType: eventTypeInstance?.eventType,
       eventSubType: eventTypeInstance?.eventSubType,
     });
+  } finally {
+    await eventTypeInstance.keepAlivePromise;
   }
 };
 
@@ -314,7 +318,7 @@ const _checkEventIsBlocked = async (baseInstance) => {
 
 const _processEvent = async (eventTypeInstance, processContext, key, queueEntries, payload) => {
   try {
-    const eventOutdated = await eventTypeInstance.isOutdatedAndKeepalive(queueEntries);
+    const eventOutdated = await eventTypeInstance.isOutdatedAndKeepAlive(queueEntries);
     if (eventOutdated) {
       // NOTE: return empty status map to comply with the interface
       return {};

package/src/shared/openTelemetry.js

@@ -1,9 +1,8 @@
 "use strict";
 
 const cds = require("@sap/cds");
-let otel, telemetry;
+let otel;
 try {
-  telemetry = require("@cap-js/telemetry");
   otel = require("@opentelemetry/api");
 } catch {
   // ignore
@@ -14,12 +13,13 @@ const config = require("../config");
 const COMPONENT_NAME = "/shared/openTelemetry";
 
 const trace = async (context, label, fn, { attributes = {}, newRootSpan = false } = {}) => {
-  if (!config.enableCAPTelemetry || !otel || !telemetry) {
+  const tracerProvider = otel?.trace.getTracerProvider();
+  // Check if a real provider is registered
+  if (!config.enableCAPTelemetry || !tracerProvider || tracerProvider === otel.trace.NOOP_TRACER_PROVIDER) {
     return fn();
   }
 
   const tracer = otel.trace.getTracer("eventqueue");
-
   const span = tracer.startSpan(`eventqueue-${label}`, {
     kind: otel.SpanKind.INTERNAL,
     root: newRootSpan,