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

GH-40799: [Doc][Format] Implementation status page should list canonical extension types #41053

Merged
merged 3 commits into from
Apr 8, 2024

Conversation

rok
Copy link
Member

@rok rok commented Apr 6, 2024

Rationale for this change

Two specifications and one implementation of canonical extension types were added and this should be documented.

What changes are included in this PR?

This represents current state of canonical extension types.

Are these changes tested?

No, docs only.

Are there any user-facing changes?

In so much as they read docs.

Copy link

github-actions bot commented Apr 6, 2024

⚠️ GitHub issue #40799 has been automatically assigned in GitHub to PR creator.

@rok rok assigned rok and unassigned rok Apr 6, 2024
@rok
Copy link
Member Author

rok commented Apr 6, 2024

@github-actions crossbow submit preview-docs

Copy link

github-actions bot commented Apr 6, 2024

Revision: ae56dec

Submitted crossbow builds: ursacomputing/crossbow @ actions-cdc4d5be63

Task Status
preview-docs GitHub Actions

@rok
Copy link
Member Author

rok commented Apr 6, 2024

Current docs, proposed docs

@rok rok requested a review from pitrou April 6, 2024 23:06
@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Apr 8, 2024
Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

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

Thanks a lot @rok , just a nit.

Comment on lines 118 to 120
| FixedShapeTensor | ✓ | | | | | | | |
+---------------------+-------+-------+-------+------------+-------+-------+-------+-------+
| VariableShapeTensor | | | | | | | | |
Copy link
Member

Choose a reason for hiding this comment

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

These are the C++ and Python internal names, should we use the prose spelling instead? "Fixed shape tensor" and "Variable shape tensor"

Copy link
Member Author

Choose a reason for hiding this comment

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

Good point, thanks for catching this! Fixed.

docs/source/status.rst Outdated Show resolved Hide resolved
@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting merge Awaiting merge labels Apr 8, 2024
docs/source/status.rst Outdated Show resolved Hide resolved
@rok rok requested a review from pitrou April 8, 2024 14:34
@github-actions github-actions bot added awaiting change review Awaiting change review awaiting changes Awaiting changes and removed awaiting changes Awaiting changes awaiting change review Awaiting change review labels Apr 8, 2024
@rok rok merged commit 072eff3 into apache:main Apr 8, 2024
8 checks passed
@rok rok deleted the GH-40799 branch April 8, 2024 18:37
@rok rok removed the awaiting changes Awaiting changes label Apr 8, 2024
Copy link

After merging your PR, Conbench analyzed the 4 benchmarking runs that have been run so far on merge-commit 072eff3.

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 2 possible false positives for unstable benchmarks that are known to sometimes produce them.

raulcd pushed a commit that referenced this pull request Apr 9, 2024
…cal extension types (#41053)

### Rationale for this change

Two specifications and one implementation of canonical extension types were added and this should be documented.

### What changes are included in this PR?

This represents current state of canonical extension types.

### Are these changes tested?

No, docs only.

### Are there any user-facing changes?

In so much as they read docs.
* GitHub Issue: #40799

Authored-by: Rok Mihevc <rok@mihevc.org>
Signed-off-by: Rok Mihevc <rok@mihevc.org>
verma-kartik pushed a commit to verma-kartik/arrow that referenced this pull request Apr 11, 2024
…canonical extension types (apache#41053)

### Rationale for this change

Two specifications and one implementation of canonical extension types were added and this should be documented.

### What changes are included in this PR?

This represents current state of canonical extension types.

### Are these changes tested?

No, docs only.

### Are there any user-facing changes?

In so much as they read docs.
* GitHub Issue: apache#40799

Authored-by: Rok Mihevc <rok@mihevc.org>
Signed-off-by: Rok Mihevc <rok@mihevc.org>
vibhatha pushed a commit to vibhatha/arrow that referenced this pull request Apr 15, 2024
…canonical extension types (apache#41053)

### Rationale for this change

Two specifications and one implementation of canonical extension types were added and this should be documented.

### What changes are included in this PR?

This represents current state of canonical extension types.

### Are these changes tested?

No, docs only.

### Are there any user-facing changes?

In so much as they read docs.
* GitHub Issue: apache#40799

Authored-by: Rok Mihevc <rok@mihevc.org>
Signed-off-by: Rok Mihevc <rok@mihevc.org>
tolleybot pushed a commit to tmct/arrow that referenced this pull request May 2, 2024
…canonical extension types (apache#41053)

### Rationale for this change

Two specifications and one implementation of canonical extension types were added and this should be documented.

### What changes are included in this PR?

This represents current state of canonical extension types.

### Are these changes tested?

No, docs only.

### Are there any user-facing changes?

In so much as they read docs.
* GitHub Issue: apache#40799

Authored-by: Rok Mihevc <rok@mihevc.org>
Signed-off-by: Rok Mihevc <rok@mihevc.org>
tolleybot pushed a commit to tmct/arrow that referenced this pull request May 4, 2024
…canonical extension types (apache#41053)

### Rationale for this change

Two specifications and one implementation of canonical extension types were added and this should be documented.

### What changes are included in this PR?

This represents current state of canonical extension types.

### Are these changes tested?

No, docs only.

### Are there any user-facing changes?

In so much as they read docs.
* GitHub Issue: apache#40799

Authored-by: Rok Mihevc <rok@mihevc.org>
Signed-off-by: Rok Mihevc <rok@mihevc.org>
rok added a commit to tmct/arrow that referenced this pull request May 8, 2024
…canonical extension types (apache#41053)

### Rationale for this change

Two specifications and one implementation of canonical extension types were added and this should be documented.

### What changes are included in this PR?

This represents current state of canonical extension types.

### Are these changes tested?

No, docs only.

### Are there any user-facing changes?

In so much as they read docs.
* GitHub Issue: apache#40799

Authored-by: Rok Mihevc <rok@mihevc.org>
Signed-off-by: Rok Mihevc <rok@mihevc.org>
rok added a commit to tmct/arrow that referenced this pull request May 8, 2024
…canonical extension types (apache#41053)

### Rationale for this change

Two specifications and one implementation of canonical extension types were added and this should be documented.

### What changes are included in this PR?

This represents current state of canonical extension types.

### Are these changes tested?

No, docs only.

### Are there any user-facing changes?

In so much as they read docs.
* GitHub Issue: apache#40799

Authored-by: Rok Mihevc <rok@mihevc.org>
Signed-off-by: Rok Mihevc <rok@mihevc.org>
vibhatha pushed a commit to vibhatha/arrow that referenced this pull request May 25, 2024
…canonical extension types (apache#41053)

### Rationale for this change

Two specifications and one implementation of canonical extension types were added and this should be documented.

### What changes are included in this PR?

This represents current state of canonical extension types.

### Are these changes tested?

No, docs only.

### Are there any user-facing changes?

In so much as they read docs.
* GitHub Issue: apache#40799

Authored-by: Rok Mihevc <rok@mihevc.org>
Signed-off-by: Rok Mihevc <rok@mihevc.org>
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.

[Doc][Format] Implementation status page should list canonical extension types
3 participants