Skip to content

Conversation

@dttung2905
Copy link
Contributor

Which issue does this PR close?

Closes #12280

Rationale for this change

One of the links in datafusion::logical_expr::UserDefinedLogicalNodeCore doc throws an invalid resource
image

What changes are included in this PR?

Referring to the file in a different module is abit tricky to I chose to refer to a remote Github file instead

Are these changes tested?

Its a doc change so no test is needed (I think)

Are there any user-facing changes?

Yes. It will change the API doc

Signed-off-by: dttung2905 <ttdao.2015@accountancy.smu.edu.sg>
@github-actions github-actions bot added the logical-expr Logical plan and expressions label Sep 9, 2024
@alamb alamb added the documentation Improvements or additions to documentation label Sep 9, 2024
Co-authored-by: Andrew Lamb <andrew@nerdnetworks.org>
@github-actions github-actions bot removed the documentation Improvements or additions to documentation label Sep 9, 2024
@alamb
Copy link
Contributor

alamb commented Sep 9, 2024

Thanks again @dttung2905 📖

@alamb alamb merged commit 23f3a8f into apache:main Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

logical-expr Logical plan and expressions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Can‘t refer example file in UserDefinedLogicalNodeCore doc

2 participants