docs: Windows PowerShell compatibility improvements across Quickstart, Install, Playground, and Dev Environment guides#360
Open
jaiswal-naman wants to merge 1 commit intodatazip-inc:masterfrom
Conversation
… and community docs - quickstart.mdx: add Windows PowerShell tabs for docker compose setup and update commands - olake-ui/index.mdx: add Windows tabs, expand port conflict troubleshooting section - docker-cli.mdx: add note on backslash vs backtick line continuation in PowerShell - playground.mdx: fix missing git clone step for all 3 query engines (Trino/Presto/Spark), add Windows tabs - setting-up-a-dev-env.mdx: add Windows tabs for all prereqs (winget), clone, discover, sync, JAR build, debug path - WINDOWS_SETUP_GUIDE.md: new standalone quick-reference guide for Windows users Closes datazip-inc#357
|
Naman Jaiswal seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
docs: Windows PowerShell compatibility improvements across Quickstart, Install, Playground, and Dev Environment guides
Closes #357
Why this PR
This PR resolves Windows-specific documentation gaps that make the current guides difficult or impossible to follow in PowerShell.
Before these changes, key commands were Unix-only (
curl | docker compose,sh -c,./build.sh,lsof,$(pwd)paths,\line continuation). As a result, Windows users either hit command errors directly or had to infer unsupported workarounds.This PR adds explicit Windows PowerShell paths while preserving all existing macOS/Linux instructions.
Scope
Files updated
docs/getting-started/quickstart.mdxdocs/install/olake-ui/index.mdxdocs/install/docker-cli.mdxdocs/getting-started/playground.mdxdocs/community/setting-up-a-dev-env.mdxChange size (from previous validated patch set)
docs/community/setting-up-a-dev-env.mdx: +246 / -3docs/getting-started/playground.mdx: +258 / -48docs/getting-started/quickstart.mdx: +77 / -18docs/install/docker-cli.mdx: +22 / -0docs/install/olake-ui/index.mdx: +159 / -26Detailed changes (with before/after)
1)
docs/getting-started/quickstart.mdxWhat changed
docker info) in prerequisites.docker-compose-v1.yml)docker-compose.yml)down+pull+up -dflow.Why
The original Unix pipeline pattern is not reliable for PowerShell users:
Before (Unix-only):
curl -sSL https://raw.githubusercontent.com/datazip-inc/olake-ui/master/docker-compose-v1.yml | docker compose -f - up -dAfter (Windows tab added):
Screenshot placeholders
docker info)down/pull/up)2)
docs/install/olake-ui/index.mdxWhat changed
Why
This page is often the first install entry point. Without PowerShell-native commands, Windows users encounter immediate friction.
Before/after highlight: Port conflict guidance
Before:
After:
Screenshot placeholders
3)
docs/install/docker-cli.mdxWhat changed
Windows Usersnote clarifying PowerShell line continuation differences.Why
PowerShell does not use
\as a line continuation character. Many users copy multi-line examples as-is and hit failures.Before/after highlight
Before (Unix-style multiline only):
docker run --pull=always \ -v "/path/to/config:/mnt/config" \ olakego/source-postgres:latest \ discover \ --config /mnt/config/source.jsonAfter (Windows equivalent shown):
Screenshot placeholders
\vs backtick4)
docs/getting-started/playground.mdxWhat changed
olakerepo (notolake-ui).Set-Locationequivalents.Why
This was a functional documentation gap: users were instructed to navigate into engine directories without first cloning the repository that contains those directories.
Before/after highlight
Before:
After:
git clone https://github.com/datazip-inc/olake.git cd olake/examples/trino-tablurarest-minio-mysql docker compose up -dWindows equivalent added:
Important naming correctness
Directory names follow upstream exactly:
trino-tablurarest-minio-mysqlpresto-tabularest-minio-mysqlspark-tablurarest-minio-mysql(Presto spelling differs from Trino/Spark and is intentionally preserved.)
Screenshot placeholders
presto-tabularest-...spark-tablurarest-...Set-Locationexamples5)
docs/community/setting-up-a-dev-env.mdxWhat changed
Comprehensive Windows support added across the full developer journey:
wingetinstall commands + verification commandsJAVA_HOMEcaution blockwsl --install)go runcommands for:netstat -ano | findstr :8888)Why
The original flow assumed Bash availability and Unix utilities. This prevented clean execution on Windows for core developer tasks.
Before/after highlights
Before (Unix-only setup shell):
sh -c 'curl ... && curl ... && docker compose ...'After (PowerShell tab):
Before (Unix-only execution):
./build.sh driver-postgres discover --config $(pwd)/source.jsonAfter (Windows native execution):
Screenshot placeholders
go run)Validation summary
Reviewer checklist
presto-tabularest-...(intentional upstream naming)Notes for maintainers
This PR is intentionally documentation-only and platform-compatibility focused. No behavioral/runtime code changes are included.