Skip to content

Conversation

@alamb
Copy link
Contributor

@alamb alamb commented Mar 21, 2023

Related to apache/datafusion#5500

We are changing datafusion docs to be published to the arrow-datafusion repo (see apache/datafusion#5670)

@alamb alamb changed the title Website: Delete datafusion directory WEBSITE: Delete datafusion directory Mar 21, 2023
@alamb alamb changed the title WEBSITE: Delete datafusion directory WEBSITE: Delete static datafusion directory Mar 21, 2023
@kou kou changed the title WEBSITE: Delete static datafusion directory Delete static datafusion directory Mar 21, 2023
@kou
Copy link
Member

kou commented Mar 21, 2023

@alamb
Copy link
Contributor Author

alamb commented Mar 22, 2023

Could you also remove https://github.com/apache/arrow-site/blob/main/.github/workflows/deploy.yml#L104-L105 ?

Done (I made a new PR as it needs to target main and this PR targets asf-site)

#337

@alamb alamb requested review from andygrove and kou March 22, 2023 13:57
kou pushed a commit that referenced this pull request Mar 22, 2023
…thon` (#337)

Per @kou 's suggestion
#336 (comment)

We are now serving datafusion content from the datafusion-repo -- see
apache/datafusion#5500
@kou
Copy link
Member

kou commented Mar 22, 2023

Thanks!

It seems that we don't need this pull request anymore because our auto deploy job removed /datafusion/ by the #337 changes: 9e27a1f

But https://arrow.apache.org/datafusion/ is broken now. I'll re-publish https://arrow.apache.org/datafusion/ by re-running https://github.com/apache/arrow-datafusion/actions/runs/4489457202 .

@kou
Copy link
Member

kou commented Mar 22, 2023

Ah, re-running the job doesn't re-publish because the job doesn't push anything when there are no changes.

@kou
Copy link
Member

kou commented Mar 22, 2023

I've merged apache/datafusion#5687 to re-publish https://arrow.apache.org/datafusion/ .

@kou
Copy link
Member

kou commented Mar 22, 2023

Ah... Changing README.md doesn't change built HTML...

@alamb
Copy link
Contributor Author

alamb commented Mar 23, 2023

Ah... Changing README.md doesn't change built HTML...

I will make another change to republish the DataFusion site

@alamb
Copy link
Contributor Author

alamb commented Mar 23, 2023

apache/datafusion#5702

@alamb
Copy link
Contributor Author

alamb commented Mar 23, 2023

apache/datafusion#5702 (comment)

We appear to be fixed now.

Closing this PR as the directory is no longer present on asf-site: https://github.com/apache/arrow-site/tree/asf-site

@alamb alamb closed this Mar 23, 2023
@alamb alamb deleted the alamb/delete_datafusion branch March 23, 2023 11:24
@kou
Copy link
Member

kou commented Mar 23, 2023

Thanks!

@alamb
Copy link
Contributor Author

alamb commented Mar 23, 2023

Thanks!

Thank YOU for all the help @kou -- it is very much appreciated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants