Enhance trigger validation in MARS-828 verification script to improve error handling and output clarity

MARS_Packages/REL01_ADDITIONS/MARS-828/05_MARS_828_verify_installation.sql

@@ -81,6 +81,8 @@ SELECT CT_MRDS.FILE_ARCHIVER.GET_VERSION() as package_version FROM DUAL;
 -- 7. Test trigger validation
 PROMPT
 PROMPT 7. Testing trigger validation (should fail)...
+WHENEVER SQLERROR CONTINUE
+SET SERVEROUTPUT ON
 DECLARE
     vTestPassed BOOLEAN := FALSE;
 BEGIN
@@ -107,14 +109,16 @@ EXCEPTION
     WHEN OTHERS THEN
         IF SQLCODE = -20999 THEN
             DBMS_OUTPUT.PUT_LINE('SUCCESS: Trigger validation working correctly');
-            DBMS_OUTPUT.PUT_LINE('Expected error: ' || SQLERRM);
+            DBMS_OUTPUT.PUT_LINE('Trigger correctly rejected MINIMUM_AGE_MONTHS strategy without required value');
             vTestPassed := TRUE;
         ELSE
-            DBMS_OUTPUT.PUT_LINE('ERROR: Unexpected error: ' || SQLERRM);
+            DBMS_OUTPUT.PUT_LINE('ERROR: Unexpected error occurred during trigger validation');
+            DBMS_OUTPUT.PUT_LINE('Error code: ' || SQLCODE);
         END IF;
         ROLLBACK;
 END;
 /
+WHENEVER SQLERROR EXIT FAILURE
 
 PROMPT
 PROMPT ========================================

MARS_Packages/REL01_ADDITIONS/MARS-828/new_version/FILE_ARCHIVER.pkb

@@ -234,7 +234,7 @@ AS
 
             -- Note: DBMS_CLOUD.EXPORT_DATA may create multiple parquet files (parallel execution)
             -- Instead of tracking individual files, we store the archive directory prefix
-            -- ARCH_FILE_NAME will contain the directory URI where all parquet files are located
+            -- ARCH_PATH contain the directory URI where all parquet files are located
             vFilename := vUri; -- Store directory prefix instead of individual filename
 
             -- Try to drop EXPORTED FILES ("regular data files")
@@ -245,7 +245,7 @@ AS
                   BEGIN
                      UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED r
                         SET PROCESSING_STATUS = 'ARCHIVED_AND_TRASHED'  -- Status reflects file is archived and kept in TRASH
-                           ,ARCH_FILE_NAME = vFilename  -- Now contains directory prefix, not individual file
+                           ,ARCH_PATH = vFilename  -- Now contains directory prefix, not individual file
                            ,PARTITION_YEAR = ym_loop.year    -- Record which partition year the data was archived to
                            ,PARTITION_MONTH = ym_loop.month  -- Record which partition month the data was archived to
                      WHERE r.a_source_file_config_key= pSourceFileConfigKey
@@ -324,7 +324,7 @@ AS
 
                         UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED r
                            SET PROCESSING_STATUS = 'INGESTED'
-                              ,ARCH_FILE_NAME = NULL
+                              ,ARCH_PATH = NULL
                         WHERE r.a_source_file_config_key = pSourceFileConfigKey
                           AND r.source_file_name = f.filename
                         ;
@@ -579,7 +579,7 @@ AS
 
          UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED
             SET PROCESSING_STATUS = 'INGESTED',
-                ARCH_FILE_NAME = NULL,
+                ARCH_PATH = NULL,
                 PARTITION_YEAR = NULL,
                 PARTITION_MONTH = NULL
           WHERE A_SOURCE_FILE_RECEIVED_KEY = pSourceFileReceivedKey
@@ -620,7 +620,7 @@ AS
 
          UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED
             SET PROCESSING_STATUS = 'INGESTED',
-                ARCH_FILE_NAME = NULL,
+                ARCH_PATH = NULL,
                 PARTITION_YEAR = NULL,
                 PARTITION_MONTH = NULL
           WHERE A_SOURCE_FILE_CONFIG_KEY = pSourceFileConfigKey
@@ -660,7 +660,7 @@ AS
 
          UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED
             SET PROCESSING_STATUS = 'INGESTED',
-                ARCH_FILE_NAME = NULL,
+                ARCH_PATH  = NULL,
                 PARTITION_YEAR = NULL,
                 PARTITION_MONTH = NULL
           WHERE PROCESSING_STATUS = 'ARCHIVED_AND_TRASHED';

MARS_Packages/REL01_ADDITIONS/MARS-828/rollback_version/v2.0.0/FILE_ARCHIVER.pkb

@@ -181,7 +181,7 @@ AS
                   BEGIN
                      UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED r
                         SET PROCESSING_STATUS = 'ARCHIVED'
-                           ,ARCH_FILE_NAME = vUri||vFilename
+                           ,ARCH_PATH = vUri||vFilename
                      WHERE r.a_source_file_config_key= pSourceFileConfigKey
                        AND r.source_file_name = f.filename
                        AND r.processing_status = 'INGESTED'
@@ -244,7 +244,7 @@ AS
 
                         UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED r
                            SET PROCESSING_STATUS = 'INGESTED'
-                              ,ARCH_FILE_NAME = NULL
+                              ,ARCH_PATH = NULL
                         WHERE r.a_source_file_config_key = pSourceFileConfigKey
                           AND r.source_file_name = f.filename
                         ;

MARS_Packages/REL01_ADDITIONS/MARS-828/rollback_version/v3.0.0/FILE_ARCHIVER.pkb

@@ -231,7 +231,7 @@ AS
                   BEGIN
                      UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED r
                         SET PROCESSING_STATUS = 'ARCHIVED'
-                           ,ARCH_FILE_NAME = vUri||vFilename
+                           ,ARCH_PATH = vUri||vFilename
                      WHERE r.a_source_file_config_key= pSourceFileConfigKey
                        AND r.source_file_name = f.filename
                        AND r.processing_status = 'INGESTED'
@@ -294,7 +294,7 @@ AS
 
                         UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED r
                            SET PROCESSING_STATUS = 'INGESTED'
-                              ,ARCH_FILE_NAME = NULL
+                              ,ARCH_PATH = NULL
                         WHERE r.a_source_file_config_key = pSourceFileConfigKey
                           AND r.source_file_name = f.filename
                         ;

MARS_Packages/REL01_ADDITIONS/MARS-828/rollback_version/v3.2.1/FILE_ARCHIVER.pkb

@@ -223,7 +223,7 @@ AS
 
             -- Note: DBMS_CLOUD.EXPORT_DATA may create multiple parquet files (parallel execution)
             -- Instead of tracking individual files, we store the archive directory prefix
-            -- ARCH_FILE_NAME will contain the directory URI where all parquet files are located
+            -- ARCH_PATH will contain the directory URI where all parquet files are located
             vFilename := vUri; -- Store directory prefix instead of individual filename
 
             -- Try to drop EXPORTED FILES ("regular data files")
@@ -234,7 +234,7 @@ AS
                   BEGIN
                      UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED r
                         SET PROCESSING_STATUS = 'ARCHIVED_AND_TRASHED'  -- Status reflects file is archived and kept in TRASH
-                           ,ARCH_FILE_NAME = vFilename  -- Now contains directory prefix, not individual file
+                           ,ARCH_PATH = vFilename  -- Now contains directory prefix, not individual file
                            ,PARTITION_YEAR = ym_loop.year    -- Record which partition year the data was archived to
                            ,PARTITION_MONTH = ym_loop.month  -- Record which partition month the data was archived to
                      WHERE r.a_source_file_config_key= pSourceFileConfigKey
@@ -313,7 +313,7 @@ AS
 
                         UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED r
                            SET PROCESSING_STATUS = 'INGESTED'
-                              ,ARCH_FILE_NAME = NULL
+                              ,ARCH_PATH = NULL
                         WHERE r.a_source_file_config_key = pSourceFileConfigKey
                           AND r.source_file_name = f.filename
                         ;
@@ -568,7 +568,7 @@ AS
 
          UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED
             SET PROCESSING_STATUS = 'INGESTED',
-                ARCH_FILE_NAME = NULL,
+                ARCH_PATH = NULL,
                 PARTITION_YEAR = NULL,
                 PARTITION_MONTH = NULL
           WHERE A_SOURCE_FILE_RECEIVED_KEY = pSourceFileReceivedKey
@@ -609,7 +609,7 @@ AS
 
          UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED
             SET PROCESSING_STATUS = 'INGESTED',
-                ARCH_FILE_NAME = NULL,
+                ARCH_PATH = NULL,
                 PARTITION_YEAR = NULL,
                 PARTITION_MONTH = NULL
           WHERE A_SOURCE_FILE_CONFIG_KEY = pSourceFileConfigKey
@@ -649,7 +649,7 @@ AS
 
          UPDATE CT_MRDS.A_SOURCE_FILE_RECEIVED
             SET PROCESSING_STATUS = 'INGESTED',
-                ARCH_FILE_NAME = NULL,
+                ARCH_PATH = NULL,
                 PARTITION_YEAR = NULL,
                 PARTITION_MONTH = NULL
           WHERE PROCESSING_STATUS = 'ARCHIVED_AND_TRASHED';

README.md

@@ -75,6 +75,6 @@ sql "ADMIN/Cloudpass#34@ggmichalski_high" "@rollback_mars1057.sql"
 
 
 cd .\MARS_Packages\REL01_ADDITIONS\MARS-828
-sql "ADMIN/Cloudpass#34@ggmichalski_high" "@install_mars828.sql"
-sql "ADMIN/Cloudpass#34@ggmichalski_high" "@rollback_mars828.sql"
+echo 'yes' | sql "ADMIN/Cloudpass#34@ggmichalski_high" "@install_mars828.sql"
+echo 'yes' | sql "ADMIN/Cloudpass#34@ggmichalski_high" "@rollback_mars828.sql"
 7z a -pMojeSuperHaslo#123 -mhe=on M828_arch.7z MARS-828\