Skip to content

Conversation

@Fokko
Copy link
Contributor

@Fokko Fokko commented Jun 13, 2022

I think we forgot to update the home page from the Parquet block.

I think we forgot to update the home page from the Parquet block.
@singhpk234
Copy link
Contributor

singhpk234 commented Jun 13, 2022

@Fokko , presently the CI ran all the ut's of engines, even for an update for license. As per my understanding it should not be required, as presently we avoid running them when we update something in readme.md etc, have filed a pr for the same, Please do let me know, your thoughts on, if it makes sense doing the same for LICENSE ?

@Fokko
Copy link
Contributor Author

Fokko commented Jun 13, 2022

@singhpk234 that makes sense, thanks!

@rdblue
Copy link
Contributor

rdblue commented Jun 13, 2022

@Fokko, can you check the other LICENSE files for binaries and update those if needed? We may have copied this more places.

@rdblue rdblue merged commit 06bd31e into apache:master Jun 13, 2022
@Fokko Fokko deleted the patch-4 branch June 13, 2022 20:42
@Fokko
Copy link
Contributor Author

Fokko commented Jun 13, 2022

@rdblue I checked the rest, and it looks good 👍🏻

namrathamyske pushed a commit to namrathamyske/iceberg that referenced this pull request Jul 10, 2022
I think we forgot to update the home page from the Parquet block.
namrathamyske pushed a commit to namrathamyske/iceberg that referenced this pull request Jul 10, 2022
I think we forgot to update the home page from the Parquet block.
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.

3 participants