code review comments incorporated

Author: ShashwatDadhich

pkg/pipeline/CiCdPipelineOrchestrator.go

@@ -1277,14 +1277,7 @@ func (impl CiCdPipelineOrchestratorImpl) CreateMaterials(createMaterialRequest *
 	if err != nil {
 		return nil, err
 	}
-	defer func() {
-		if err != nil {
-			err := tx.Rollback()
-			if err != nil {
-				impl.logger.Errorw("error in rollback Create material", "err", err)
-			}
-		}
-	}()
+	defer tx.Rollback()
 	existingMaterials, err := impl.materialRepository.FindByAppId(createMaterialRequest.AppId)
 	if err != nil {
 		impl.logger.Errorw("err", "err", err)
@@ -1335,14 +1328,7 @@ func (impl CiCdPipelineOrchestratorImpl) UpdateMaterial(updateMaterialDTO *bean.
 	if err != nil {
 		return nil, err
 	}
-	defer func() {
-		if err != nil {
-			err := tx.Rollback()
-			if err != nil {
-				impl.logger.Errorw("error in rollback Update material", "err", err)
-			}
-		}
-	}()
+	defer tx.Rollback()
 	updatedMaterial, err := impl.updateMaterial(updateMaterialDTO, tx)
 	if err != nil {
 		impl.logger.Errorw("err", "err", err)

pkg/pipeline/CiMaterialConfigService.go

@@ -129,14 +129,8 @@ func (impl *CiMaterialConfigServiceImpl) DeleteMaterial(request *bean.UpdateMate
 	if err != nil {
 		return err
 	}
-	defer func() {
-		if err != nil {
-			err := tx.Rollback()
-			if err != nil {
-				impl.logger.Errorw("error in rollback ", "err", err)
-			}
-		}
-	}()
+
+	defer tx.Rollback()
 
 	err = impl.materialRepo.MarkMaterialDeleted(existingMaterial, tx)
 	if err != nil {