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

DRAFT: merging #143

Closed
wants to merge 7 commits into from
Closed

Conversation

elijahpetty
Copy link

@elijahpetty elijahpetty self-assigned this Jan 19, 2024
OUTLINE-merging.md Outdated Show resolved Hide resolved
docs/working-with-deephaven-tables/merging-tables.md Outdated Show resolved Hide resolved
docs/working-with-deephaven-tables/merging-tables.md Outdated Show resolved Hide resolved
sidebar_label: Merging tables in Deephaven
---

This guide discusses how to merge tables in Deephaven.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm guessing we want an explanation of why to choose a merge vs. a join, and so the text of the note can be included in that brief introduction.

docs/working-with-deephaven-tables/merging-tables.md Outdated Show resolved Hide resolved
docs/working-with-deephaven-tables/merging-tables.md Outdated Show resolved Hide resolved
@elijahpetty elijahpetty changed the title OUTLINE: merging DRAFT: merging Jan 26, 2024

![img](../assets/working-with-deephaven-tables/merge-n-sort-vs-merge-sorted.png)

When we use `merge_sorted`, our query completes ten times as fast as it does when using `merge` followed by `sort`.
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a good amount of variation from one run to the next here. I will need to run this multiple times and get the average for this to be a meaningful and honest figure.

@elijahpetty elijahpetty deleted the merging-outline branch March 15, 2024 16:07
@github-actions github-actions bot locked and limited conversation to collaborators Mar 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants