Introduce duckdb-wasm and apache-arrow#2
Merged
Merged
Conversation
hackerwins
force-pushed
the
duckdb
branch
2 times, most recently
from
May 11, 2024 09:16
98cea9f to
0ca79ec
Compare
5 tasks
5 tasks
hackerwins
added a commit
that referenced
this pull request
May 9, 2026
Marks lesson #2 as no longer latent; T12 ghost text was the trigger that turned the dropped-field bug into a live render miss, and the fix is one yorkieToPlain unwrap per element branch. The new lesson cites the audit rule for future agents: new element fields need both write (addSlide) AND read (read()) propagation, MemSlidesStore parity tests miss it. Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
hackerwins
added a commit
that referenced
this pull request
May 9, 2026
Marks every step in the todo file as completed and moves the todo + lessons pair into docs/tasks/archive/2026/05/ so docs/tasks/active/ shows only in-flight work. The lessons file's #2 entry was already updated mid-PR when the read() placeholderRef stripping turned out to be live, not latent — that record carries forward into the archive. Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
hackerwins
added a commit
that referenced
this pull request
May 24, 2026
Fills in the paired lessons file with measured WCAG contrasts, the factory-default file-discovery surprise, and follow-up signals for PR #2. The roadmap doc now uses fully qualified package paths to keep verify:entropy doc-staleness checks green, and the Status table flips PR #1 to "Ready to merge". Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
hackerwins
added a commit
that referenced
this pull request
Jul 5, 2026
Three findings from the PR review: - knownCaches strongly retained every per-measurer cache map for the module lifetime (one leaked pair per disposed editor), defeating the WeakMaps. Add disposeMeasureCache(measurer) and call it from editor.dispose(); export it from the package index. - loadingdone is unreliable on WebKit/Safari, so also settle via the Promise-based document.fonts.ready when fonts are still loading at mount (guarded on dispose). Mid-session Safari picks still rely on loadingdone. - Remove a stale 'Still open' task-log entry for the already-completed #2. Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]> Claude-Session: https://claude.ai/code/session_015EPywatWADMR2iYrwauVNL
harrykim8672
added a commit
to harrykim8672/wafflebase
that referenced
this pull request
Jul 21, 2026
mark-ready gate wafflebase#1 parsed the <!-- harness-verification --> PR comment, which the author agent could post itself (issues:write). It now reads the authoritative CI workflow-run conclusion for the head SHA via the Actions API (the author cannot forge a CI run), matching gate wafflebase#2's trust model. Added actions:read to the promote job. Also documented that agent-independent-review must never be sufficient-for- merge on its own — human CODEOWNER approval stays required (LLM-injection residual backstop). Co-Authored-By: Claude Opus 4.8 <[email protected]>
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
1 participant
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.
What this PR does / why we need it?
Experiment with DuckDB Integration and Apache Arrow for Preventing Layered Data Copy
This PR focuses on conducting an experiment to prevent layered data copy by integrating DuckDB with Apache Arrow.
Any background context you want to provide?
Initially, we attempted to upload the CVS files by loading them into memory, leading to inefficiencies in data processing as highlighted below:
Results:
For the second phase, we plan to upload the CSV files directly as test files, bypassing the need for loading into memory, as illustrated by:
Reference: DuckDB CSV Import Guide
Key Findings:
What are the relevant tickets?
Fixes:
Checklist