GIScience / GIScience/ohsome-planet

improve error message if parquet target exists

Open Beginner friendly
#48 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Java
Stars
61
Forks
7
PR merge metrics
No merged PRs in 30d

Description

The current error message does not tell you which parquet directory is not empty. If the non-empty path would be shown, the error message would be more helpful.

https://github.com/GIScience/ohsome-planet/blob/319c229cd83a49cbebc8567aacbdc63bac6769b8/ohsome-planet-cli/src/main/java/org/heigit/ohsome/planet/cmd/Contributions.java#L135

Contributor guide

No contributing guide indexed for this repository

Research direction

Read ohsome-planet-cli/src/main/java/org/heigit/ohsome/planet/cmd/Contributions.java around line 135 and trace the check for an existing parquet target. Update the error wording so it identifies the non-empty parquet directory, then verify the message when the target already contains files.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
cli
Issue type
Bug
Difficulty
1/5
Estimated time
Under an hour
Activity status
Quiet
Clarity
Clearly specified
Newbie friendliness
78/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.