-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
GH-41673: [Format][Docs] Add arrow format introductory page #41593
Conversation
cc @amoeba this could use a look already. I think all I wanted to add is here. Will need to do a general look through one more time before marking it ready for review though. |
@github-actions crossbow submit preview-docs |
Revision: 3cdd97a Submitted crossbow builds: ursacomputing/crossbow @ actions-4a1cc2326d
|
docs/source/format/index.rst
Outdated
CDataInterface | ||
CStreamInterface | ||
CDeviceDataInterface |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jorisvandenbossche have kept C Stream Interface in a separate file as the structure is nicer IMHO.
803b4b9
to
4d2bf8a
Compare
@github-actions crossbow submit preview-docs |
Revision: 4d2bf8a Submitted crossbow builds: ursacomputing/crossbow @ actions-cc7da250f4
|
Update: I have removed the change in docs/source/format/index.rst (captions for the Specifications section) and will move it to a separate PR, see 97e4217. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just had some minor tweaks from a final read-through. Thank you for taking this on!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I went through all the figures this time.
Co-authored-by: Bryce Mecum <[email protected]>
Co-authored-by: Bryce Mecum <[email protected]>
Thank you all for reviewing this PR, not a small chunk of content! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
just a minor grammar nit I found but looks great! Thanks @AlenkaF
Co-authored-by: Raúl Cumplido <[email protected]>
@github-actions crossbow submit preview-docs |
Revision: 158ee32 Submitted crossbow builds: ursacomputing/crossbow @ actions-a04a6c8710
|
I am not sure why the link to the preview is not loading http://crossbow.voltrondata.com/pr_docs/41593? @assignUser is it just me? I build the docs locally to check the html version before I merge. |
It seems to be loading for me now |
Still not working for me though 🤷♀️ |
After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit fa6a003. There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Rationale for this change
The documentation for Arrow Format could be improved:
What changes are included in this PR?
This PR includes:
in a separate "introduction" page with no technical details. Specifications index page is also restructured to include captions and make the left sidebar menu better organised.
Note: a table with all types listed together with their physical layout will be added in a separate PR to existing Columnar.rst page: #14752
Are these changes tested?
No, this is a docs change.
Are there any user-facing changes?
No.