Re: [PR] Remove staging [parquet-site]

2024-05-15 Thread via GitHub
alamb commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2112691416 > Ah okay, so it does seem like we need to clear out the output directory of `asf-staging`: https://parquet.staged.apache.org/ I suggest we make a PR that directly targets that branc

Re: [PR] Remove staging [parquet-site]

2024-05-15 Thread via GitHub
vinooganesh commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2112682297 Ah okay, so it does seem like we need to clear out the output directory of `asf-staging`: https://parquet.staged.apache.org/ -- This is an automated message from the Apache Git Ser

Re: [PR] Remove staging [parquet-site]

2024-05-15 Thread via GitHub
wgtmac merged PR #58: URL: https://github.com/apache/parquet-site/pull/58 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@parquet.apache

Re: [PR] Remove staging [parquet-site]

2024-05-15 Thread via GitHub
wgtmac commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2112664342 Sure, let me merge this. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comme

Re: [PR] Remove staging [parquet-site]

2024-05-15 Thread via GitHub
vinooganesh commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2112458273 I think we can merge as is, if only to see what the behavior will be. It seems pretty low risk, especially since we still have the branches and can revert if needed. -- This is a

Re: [PR] Remove staging [parquet-site]

2024-05-15 Thread via GitHub
wgtmac commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2112333016 > > I think I can delete the staging branch. Before that, should we send a notice to the dev ML in case there is any objection? Maybe we can set a deadline and proceed after that. I don't

Re: [PR] Remove staging [parquet-site]

2024-05-14 Thread via GitHub
wgtmac commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2111443901 cc @gszadovszky @julienledem -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific

Re: [PR] Remove staging [parquet-site]

2024-05-13 Thread via GitHub
vinooganesh commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2108802214 I initially built this to publish to the `output/` dir per https://github.com/apache/infrastructure-asfyaml/tree/main?tab=readme-ov-file#pelican-sub-directories-for-static-output. se

Re: [PR] Remove staging [parquet-site]

2024-05-13 Thread via GitHub
alamb commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2107096367 > I think I can delete the staging branch. Before that, should we send a notice to the dev ML in case there is any objection? Maybe we can set a deadline and proceed after that. I don't th

Re: [PR] Remove staging [parquet-site]

2024-05-12 Thread via GitHub
wgtmac commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2106471367 I think I can delete the staging branch. Before that, should we send a notice to the dev ML in case there is any objection? Maybe we can set a deadline and proceed after that. I don't thi

Re: [PR] Remove staging [parquet-site]

2024-05-12 Thread via GitHub
alamb commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2106397019 > The main thing I'm curious about is whether the PMC can delete branches easily from github. If so, it maybe much more straightforward, otherwise will have to file INFRA tickets In

Re: [PR] Remove staging [parquet-site]

2024-05-12 Thread via GitHub
vinooganesh commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2106379277 Yep, there is actually a sequencing of things that need to happen here: 1. Deleting the `asf-staging` brach 2. Deleting the `staging branch` 3. Deleting the README from the p

Re: [PR] Remove staging [parquet-site]

2024-05-12 Thread via GitHub
alamb commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2106376324 We may also want to update the readme too: https://github.com/apache/parquet-site/blob/production/README.md -- This is an automated message from the Apache Git Service. To respond to the

Re: [PR] Remove staging [parquet-site]

2024-05-12 Thread via GitHub
vinooganesh commented on PR #58: URL: https://github.com/apache/parquet-site/pull/58#issuecomment-2106372596 cc @wgtmac @gszadovszky @alamb after conversation on https://github.com/apache/parquet-site/pull/56 -- This is an automated message from the Apache Git Service. To respond to the

[PR] Remove staging [parquet-site]

2024-05-12 Thread via GitHub
vinooganesh opened a new pull request, #58: URL: https://github.com/apache/parquet-site/pull/58 There still needs to be an infra ticket filed to actually delete the `staging` branch (unless a PMC member can delete the branch) -- This is an automated message from the Apache Git Service. To