Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Wiki article that references new Zed join docs #1729

Merged
merged 6 commits into from
Jul 7, 2021
Merged

Conversation

philrz
Copy link
Contributor

@philrz philrz commented Jul 2, 2021

Since Brim v0.25.0 won't yet directly use the full Zed Lake API (#1618), the tl;dr on join is that the common multi-Pool joins will have to be done from outside the app via zapi for now. Now that we have the join docs in the Zed repo, the new article added in this PR points to them and walks through an example that uses zapi for such a join, after which the data is pushed back into a Pool for further use within Brim.

If you want to see how this doc will look rendered in a GitHub wiki, I've published it at a temporary location at https://github.com/philrz/scratchwiki/wiki.

@philrz philrz requested review from mccanne, nwt and a team July 2, 2021 00:25
@philrz philrz self-assigned this Jul 2, 2021
@jameskerr
Copy link
Member

Reads great to me @philrz . It reminds me that the app will need some new designs to take advantage of multiple inputs and from multi line scripts.

docs/Joining-Data-(v0.25.0+).md Outdated Show resolved Hide resolved
docs/Joining-Data-(v0.25.0+).md Outdated Show resolved Hide resolved
docs/Joining-Data-(v0.25.0+).md Outdated Show resolved Hide resolved
docs/Joining-Data-(v0.25.0+).md Outdated Show resolved Hide resolved
philrz and others added 4 commits July 6, 2021 17:52
Co-authored-by: Noah Treuhaft <noah.treuhaft@gmail.com>
Co-authored-by: Noah Treuhaft <noah.treuhaft@gmail.com>
Co-authored-by: Noah Treuhaft <noah.treuhaft@gmail.com>
@philrz philrz merged commit b637804 into main Jul 7, 2021
@philrz philrz deleted the join-doc-v0.25.0 branch July 7, 2021 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants