Skip to content

Commit b5d361d

Browse files
Merge branch 'main' into feature/data-quality-check-impact
2 parents 5c1e40c + cfd71e8 commit b5d361d

222 files changed

Lines changed: 7810 additions & 1747 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/airflow-apis-tests.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -61,6 +61,7 @@ jobs:
6161
with:
6262
ref: ${{ github.event.pull_request.head.sha }}
6363
fetch-depth: 0
64+
filter: blob:none
6465

6566
- name: Set up JDK 21
6667
uses: actions/setup-java@v4

.github/workflows/openmetadata-service-unit-tests.yml

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -74,11 +74,6 @@ jobs:
7474
uses: actions/checkout@v4
7575
with:
7676
ref: ${{ github.event_name == 'pull_request' && github.event.pull_request.head.sha || github.sha }}
77-
fetch-depth: 0
78-
79-
- name: Fetch PR base branch
80-
if: ${{ github.event_name == 'pull_request' }}
81-
run: git fetch --no-tags origin ${{ github.event.pull_request.base.ref }}
8277

8378
- name: Cache Maven dependencies
8479
uses: actions/cache@v4

.github/workflows/py-operator-build-test.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ jobs:
6666
uses: actions/checkout@v4
6767
with:
6868
ref: ${{ github.event.pull_request.head.sha }}
69-
fetch-depth: 0
7069

7170
- name: Setup Openmetadata Test Environment
7271
uses: ./.github/actions/setup-openmetadata-test-environment

.github/workflows/py-sonarcloud-nightly.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,6 @@ jobs:
4949
uses: actions/checkout@v4
5050
with:
5151
ref: ${{ env.BRANCH_NAME }}
52-
fetch-depth: 0
5352

5453
- name: Setup Openmetadata Test Environment
5554
uses: ./.github/actions/setup-openmetadata-test-environment
@@ -110,7 +109,6 @@ jobs:
110109
uses: actions/checkout@v4
111110
with:
112111
ref: ${{ env.BRANCH_NAME }}
113-
fetch-depth: 0
114112

115113
- name: Setup Openmetadata Test Environment
116114
uses: ./.github/actions/setup-openmetadata-test-environment
@@ -154,6 +152,7 @@ jobs:
154152
with:
155153
ref: ${{ env.BRANCH_NAME }}
156154
fetch-depth: 0
155+
filter: blob:none
157156

158157
- name: Setup Python
159158
uses: actions/setup-python@v5

.github/workflows/py-tests.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -232,6 +232,7 @@ jobs:
232232
with:
233233
ref: ${{ github.event.pull_request.head.sha }}
234234
fetch-depth: 0
235+
filter: blob:none
235236

236237
- name: Setup Python
237238
uses: actions/setup-python@v5

.github/workflows/ui-checkstyle.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,6 +84,7 @@ jobs:
8484
with:
8585
ref: ${{ github.event.pull_request.head.sha }}
8686
fetch-depth: 0
87+
filter: blob:none
8788

8889
- uses: actions/setup-node@v4
8990
with:
@@ -194,6 +195,7 @@ jobs:
194195
with:
195196
ref: ${{ github.event.pull_request.head.sha }}
196197
fetch-depth: 0
198+
filter: blob:none
197199

198200
- uses: actions/setup-node@v4
199201
with:
@@ -377,7 +379,6 @@ jobs:
377379
- uses: actions/checkout@v4
378380
with:
379381
ref: ${{ github.event.pull_request.head.sha }}
380-
fetch-depth: 0
381382

382383
- uses: actions/setup-node@v4
383384
with:
@@ -461,7 +462,6 @@ jobs:
461462
- uses: actions/checkout@v4
462463
with:
463464
ref: ${{ github.event.pull_request.head.sha }}
464-
fetch-depth: 0
465465

466466
- uses: actions/setup-node@v4
467467
with:
@@ -546,6 +546,7 @@ jobs:
546546
with:
547547
ref: ${{ github.event.pull_request.head.sha }}
548548
fetch-depth: 0
549+
filter: blob:none
549550

550551
- uses: actions/setup-node@v4
551552
with:
@@ -654,6 +655,7 @@ jobs:
654655
with:
655656
ref: ${{ github.event.pull_request.head.sha }}
656657
fetch-depth: 0
658+
filter: blob:none
657659

658660
- uses: actions/setup-node@v4
659661
with:
@@ -755,6 +757,7 @@ jobs:
755757
with:
756758
ref: ${{ github.event.pull_request.head.sha }}
757759
fetch-depth: 0
760+
filter: blob:none
758761

759762
- uses: actions/setup-node@v4
760763
with:

.github/workflows/yarn-coverage.yml

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,8 +48,11 @@ jobs:
4848
- uses: actions/checkout@v4
4949
with:
5050
ref: ${{ github.event.pull_request.head.sha }}
51-
# Disabling shallow clone is recommended for improving relevancy of reporting
51+
# Disabling shallow clone is recommended for improving relevancy of reporting.
52+
# Use partial clone (blob:none) to avoid downloading every blob in history —
53+
# commits/trees still available for Sonar blame; blobs fetched lazily as needed.
5254
fetch-depth: 0
55+
filter: blob:none
5356

5457
- name: Setup Node.js
5558
uses: actions/setup-node@v4

CLAUDE.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -453,6 +453,7 @@ These checks run automatically in CI. Code that violates them **will not merge**
453453
- **Keep connector-specific logic in connector-specific files**, not in generic/shared files like `builders.py`
454454
- Example: Redshift IAM auth should be in `ingestion/src/metadata/ingestion/source/database/redshift/connection.py`, not in `ingestion/src/metadata/ingestion/connections/builders.py`
455455
- This keeps the codebase modular and prevents generic utilities from becoming cluttered with connector-specific edge cases
456+
- **Use `model_str()` for Pydantic RootModel to string conversion** — OpenMetadata schema types like `ColumnName`, `EntityName`, `FullyQualifiedEntityName`, and `UUID` are Pydantic `RootModel[str]` subclasses where `str()` returns `"root='value'"` instead of the raw value. Always use `model_str()` from `metadata.ingestion.ometa.utils` instead of manual `hasattr(x, "root")` / `str(x.root)` checks.
456457

457458
### Testing Philosophy
458459
- **Test real behavior, not mock wiring** - if a test requires mocking 3+ classes just to verify a method call, it's testing the wrong thing

bin/openmetadata-server-start.sh

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,10 @@ fi
105105
if [ -z "$OPENMETADATA_JVM_PERFORMANCE_OPTS" ]; then
106106
export OPENMETADATA_JVM_PERFORMANCE_OPTS="\
107107
-server -XX:+UseG1GC \
108+
-XX:+UnlockExperimentalVMOptions \
108109
-XX:MaxGCPauseMillis=200 \
110+
-XX:G1NewSizePercent=5 \
111+
-XX:G1MaxNewSizePercent=20 \
109112
-XX:InitiatingHeapOccupancyPercent=45 \
110113
-XX:+ExplicitGCInvokesConcurrent \
111114
-XX:+UseStringDeduplication \

bin/openmetadata.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,7 +153,7 @@ fi
153153

154154
# JVM performance options
155155
if [[ -z "${CATALOG_JVM_PERF_OPTS}" ]]; then
156-
CATALOG_JVM_PERF_OPTS="-server -XX:+UseG1GC -XX:MaxGCPauseMillis=20 -XX:InitiatingHeapOccupancyPercent=35 -XX:+ExplicitGCInvokesConcurrent -Djava.awt.headless=true"
156+
CATALOG_JVM_PERF_OPTS="-server -XX:+UseG1GC -XX:+UnlockExperimentalVMOptions -XX:MaxGCPauseMillis=200 -XX:G1NewSizePercent=5 -XX:G1MaxNewSizePercent=20 -XX:InitiatingHeapOccupancyPercent=35 -XX:+ExplicitGCInvokesConcurrent -Djava.awt.headless=true"
157157
fi
158158

159159

0 commit comments

Comments
 (0)