babelfish-for-postgresql / babelfish_extensions / 16478906721

76%

BABEL_5_X_DEV: 77%

push

github

Introduce additional failure mechanism for broken views as safety check (#3976)

This change enhances the broken view handling mechanism by adding an additional layer of protection. While our primary view repair mechanism should catch and fail when attempting to repair broken views (with errors like "relation does not exist"), this change adds a failsafe mechanism to ensure broken views always fail explicitly rather than potentially returning NULL values. The expected scenario is that this view function will never ever be reached during execution but if due to some uncertainty it does then this will guarantee error instead of incorrect results.

This PR includes following changes:

1. Added new system function sys.babelfish_broken_view_function() in sys_functions.sql that raises an explicit error
2. Updated create_dummy_view_query_for_broken_view() to include a WHERE clause that triggers this function
3. Updated is_dummy_view() to recognize dummy view should have above WHERE func() = 1 in it's query

Issues Resolved: BABEL - 5974
Signed-off: Rahul Parande rparande@amazon.com

20 of 20 new or added lines in 1 file covered. (100.0%)

49690 of 65471 relevant lines covered (75.9%)

326971.09 hits per line