Skip to content

Cache label identifier data to eliminate redundant parsing #1607

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

Open
jordanpadams wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Cache label identifier data to eliminate redundant parsing #1607

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
bddab44
Cache label identifier data to eliminate redundant parsing (#1568)
jordanpadams May 19, 2026
567293e
Complete cache optimization: eliminate disk re-parse in referential i…
jordanpadams May 19, 2026
7267615
Merge branch 'main' into issues/1568-cache-label-identifiers
jordanpadams May 26, 2026
ab68a7d
Merge branch 'main' into issues/1568-cache-label-identifiers
jordanpadams May 26, 2026
dc9099e
Merge branch 'main' into issues/1568-cache-label-identifiers
jordanpadams May 28, 2026
6d9b778
Add benchmark script for validate caching performance
jordanpadams May 19, 2026
505cf89
Fix typo
jordanpadams May 28, 2026
4c31b8f
Fix caching to preserve backwards-compatible carriage-return error re…
jordanpadams May 29, 2026
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
2 changes: 1 addition & 1 deletion .github/dependabot.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ updates:
groups:
github-actions:
patterns:
- "*"kly"
- "*"

- package-ecosystem: "docker"
directory: "/docker/"
Expand Down
6 changes: 3 additions & 3 deletions CLAUDE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,10 +23,10 @@ mvn package site
# Build without running tests (useful when tests fail with extra logging)
mvn clean package -DskipTests

# Run tests only
# Run full test suite (always use this to verify changes)
mvn test

# Run tests excluding ReferenceIntegrityTest with specific Cucumber tags
# Run a targeted subset during development (NOT for verifying changes — excludes ReferenceIntegrityTest)
mvn test -Dtest=\!ReferenceIntegrityTest* -Dcucumber.filter.tags='@v3.7.x'

# Run a single Cucumber scenario by name (issue number and subtest, e.g. #1458-1)
Expand Down Expand Up @@ -172,7 +172,7 @@ Alternatively, use `--verbose 1` flag for INFO level output in reports (default
- Source/Target: Java 17 (pom.xml:172-173)

### Main Dependencies
- **pds4-jparser** (version 2.11.0) - PDS4 label parsing and validation
- **pds4-jparser** (version 3.2.0-SNAPSHOT) - PDS4 label parsing and validation; **source is at `../pds4-jparser`** — check there first when tracing errors from pds4-jparser classes (e.g. `FieldValueValidator`, `ProblemType`, parser internals)
- **pds3-product-tools** (version 4.4.2) - PDS3 validation
- **Saxon-HE** (version 12.9) - XSLT and XPath processing
- **registry-common** (version 2.1.0-SNAPSHOT) - Registry API integration
Expand Down
Loading