address review comments

Author: sid-acryl

metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py

@@ -908,21 +908,28 @@ def _is_allowed_project(self, project: TableauProject) -> bool:
         return is_allowed
 
     def _is_denied_project(self, project: TableauProject) -> bool:
-        # Either project_pattern or project_path_pattern is set
-        deny_patterns: List[str] = self.config.project_pattern.deny
-        deny_patterns.extend(self.config.project_path_pattern.deny)
+        # Either project_pattern or project_path_pattern is set in a recipe
+        # TableauConfig.projects_backward_compatibility ensures that at least one of these properties is configured.
 
-        for deny_pattern in deny_patterns:
-            # Either name or project path is denied
+        # for backward compatibility check deny list of project_pattern
+        for deny_pattern in self.config.project_pattern.deny:
             if re.match(
                 deny_pattern, project.name, self.config.project_pattern.regex_flags
-            ) or re.match(
+            ):
+                return True
+
+        for deny_pattern in self.config.project_path_pattern.deny:
+            if re.match(
                 deny_pattern,
                 self._get_project_path(project),
                 self.config.project_pattern.regex_flags,
             ):
                 return True
-        logger.info(f"project({project.name}) is not denied as per project_pattern")
+
+        logger.info(
+            f"project({project.name}) is not denied as per project_pattern(or project_path_pattern)"
+        )
+
         return False
 
     def _init_tableau_project_registry(self, all_project_map: dict) -> None: