Skip to content

Comments

Nested columns tutorial#13718

Closed
writer-jill wants to merge 4 commits intoapache:masterfrom
writer-jill:nested-tutorial
Closed

Nested columns tutorial#13718
writer-jill wants to merge 4 commits intoapache:masterfrom
writer-jill:nested-tutorial

Conversation

@writer-jill
Copy link
Contributor

Jupyter notebook tutorial on working with nested columns

This PR has:

  • been self-reviewed.

Copy link
Contributor

@317brian 317brian left a comment

Choose a reason for hiding this comment

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

This is great! Really shows what you can do with nested columns. A few minor suggestions/changes

@writer-jill writer-jill requested a review from 317brian January 30, 2023 14:01
@writer-jill writer-jill marked this pull request as ready for review January 30, 2023 14:01
Copy link
Contributor

@317brian 317brian left a comment

Choose a reason for hiding this comment

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

2 more minor things

Copy link
Contributor

@317brian 317brian left a comment

Choose a reason for hiding this comment

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

🥳 but your beginning disappeared with the apache license and intro stuff, so add that back and we're good.

@github-actions
Copy link

This pull request has been marked as stale due to 60 days of inactivity.
It will be closed in 4 weeks if no further activity occurs. If you think
that's incorrect or this pull request should instead be reviewed, please simply
write any comment. Even if closed, you can still revive the PR at any time or
discuss it on the dev@druid.apache.org list.
Thank you for your contributions.

@github-actions github-actions bot added the stale label May 12, 2023
@techdocsmith
Copy link
Contributor

@writer-jill we can use the Kafka notebook as a template to get this one fixed & into the docs

@writer-jill
Copy link
Contributor Author

Stale PR - closing & will open a new one for this tutorial.

@writer-jill writer-jill closed this Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants