[Bug] CI build script accepts flags as values for required options

Offen

#293 erstellt am 08.9.2026 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 3/5
Geschätzter Aufwand: 1-2 Tage
Anfängerfreundlichkeit: 74/100

Issue-Typ: Bug
Klarheit: Klar beschrieben
Aktivitätsstatus: Aktiv
Tech-Stack: cmake, shell
Bereich: build-system, cli

Rechercherichtung

Beginne mit ci/scripts/build_paimon.sh und führe die aufgeführten Befehle erneut aus, um nachzuverfolgen, wie erforderliche Optionswerte und --build_type validiert werden. Füge Tests auf Skriptebene für fehlende oder wie Optionen aussehende Werte sowie nicht unterstützte Build-Typen hinzu; als erledigt gilt die Aufgabe, wenn ungültige Argumente fehlschlagen, bevor Build-Verzeichnisse erstellt oder CMake aufgerufen wird, während die vier aufgeführten CMake-Build-Typen weiterhin akzeptiert werden.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

bug

Search before asking

I searched in the issues and found nothing similar.

Paimon-cpp version

Paimon version

Current main branch.

Minimal reproduce step

Run one of the following commands:

ci/scripts/build_paimon.sh --source_dir --enable_asan

ci/scripts/build_paimon.sh \
  --source_dir . \
  --build_type --enable_asan

ci/scripts/build_paimon.sh \
  --source_dir . \
  --lint_git_target_commit --enable_asan

The script treats the following option as the value of the preceding option and continues until a later command fails or CMake is invoked with an invalid value.

The script also accepts unsupported build types:

ci/scripts/build_paimon.sh \
  --source_dir . \
  --build_type Profile

What doesn't meet your expectations?

Options that require a value should reject an empty value or another --option with a clear error message.

The --build_type option should only accept the standard CMake build types used by the project:

Debug
Release
RelWithDebInfo
MinSizeRel
Invalid arguments should be rejected before creating build directories or invoking CMake.

The argument validation behavior is currently not covered by dedicated script-level tests.

Anything else?

No response

Are you willing to submit a PR?

I'm willing to submit a PR!

Vorherrschende Sprache: C++
Sterne: 65
Forks: 29
Ø Merge: 2 T. 30 Min.
Gemergte PRs (30 T.): 77

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus apache/paimon-cpp

[Feature] dictionary encoded binary support Offen

enhancement

apache/paimon-cpp#381 · 1 zugewiesene Person · Erstellt am 21.9.2026
[Feature] Bypass manifest lists on exact snapshot live-entry cache hits Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 30/100

apache/paimon-cpp#375 · 1 zugewiesene Person · Erstellt am 19.9.2026
[Feature] Access table data with the temporary credentials issued by the REST catalog Offen

enhancement

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 30/100

apache/paimon-cpp#369 · 1 zugewiesene Person · Erstellt am 17.9.2026
[Feature] Support vector and full-text search in File Index Offen

enhancement

Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 45/100

apache/paimon-cpp#361 · 1 zugewiesene Person · Erstellt am 16.9.2026
[Bug] Configured Variant shredding schema without field IDs is rejected unlike Java Paimon Offen

bug

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 68/100

apache/paimon-cpp#347 · 1 zugewiesene Person · Erstellt am 15.9.2026

Alle Issues in apache/paimon-cpp