Skip to content

CDS extractor : Implement retry for CDS compilation tasks #209

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Aug 4, 2025
Merged

CDS extractor : Implement retry for CDS compilation tasks #209

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
c5cdb06
Implement retry for CDS compilation tasks
data-douser Jul 28, 2025
9c01e2b
Enhancements for CDS compilation retry
data-douser Jul 30, 2025
08961e9
Fixes for CDS extractor compilation task retry
data-douser Jul 30, 2025
9bb6169
More fixes for CDS compilation task retry
data-douser Jul 30, 2025
2c3fa4f
Rework CDS project-only compilation & CDL.qll
data-douser Jul 31, 2025
ce36be3
Use absolute path for CdlObject in CDL.qll
data-douser Jul 31, 2025
b3520de
CDS extractor special case for index.cds
data-douser Jul 31, 2025
c5c35d7
Enforce min version for @sap/cds-dk dependency
data-douser Aug 1, 2025
0c8be00
Update CDS extractor unit tests actions workflow
data-douser Aug 1, 2025
3058ac2
Set cache-dependency-path for actions/setup-node@v4
data-douser Aug 1, 2025
6c4a8fa
CDS extractor constant for model.cds.json filename
data-douser Aug 1, 2025
3b58942
Update CdlService getImplementation in CDL.qll
data-douser Aug 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions extractors/cds/tools/cds-extractor.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ import {
logPerformanceTrackingStop,
setSourceRootDirectory,
} from './src/logging';
import { installDependencies } from './src/packageManager';
import { cacheInstallDependencies } from './src/packageManager';
import { validateArguments } from './src/utils';

// Validate the script arguments.
Expand Down Expand Up @@ -99,7 +99,7 @@ try {
for (const [projectDir, project] of dependencyGraph.projects.entries()) {
cdsExtractorLog(
'info',
`Project: ${projectDir}, Status: ${project.status}, CDS files: ${project.cdsFiles.length}, Compilations to run: ${project.cdsFilesToCompile.length}`,
`Project: ${projectDir}, Status: ${project.status}, CDS files: ${project.cdsFiles.length}, Compilation targets: ${project.compilationTargets.length}`,
);
}
} else {
Expand Down Expand Up @@ -151,7 +151,7 @@ try {
}

logPerformanceTrackingStart('Dependency Installation');
const projectCacheDirMap = installDependencies(dependencyGraph, sourceRoot, codeqlExePath);
const projectCacheDirMap = cacheInstallDependencies(dependencyGraph, sourceRoot, codeqlExePath);
logPerformanceTrackingStop('Dependency Installation');

// Check if dependency installation resulted in any usable project mappings
Expand Down
Loading
Loading