Skip to content

Conversation

@dmatrix
Copy link
Contributor

@dmatrix dmatrix commented Oct 18, 2025

  • Removed deadlinks from events/conferences
  • spark-summit.org website archives no longer accessible, resulting in a 404 error page; not a good look for the page
  • replaced with a playlist of the most recordings from previous Spark Summit years (2013-2017)

…ist recordings from previous spark summits.

Signed-off-by: Jules Damji <[email protected]>
@dmatrix dmatrix changed the title removed deadlinks from the events/conferences; replaced with videoplay list from early Spark Summit removed deadlinks from the events/conferences; replaced with videoplay list from early Spark Summits Oct 18, 2025
@dmatrix dmatrix changed the title removed deadlinks from the events/conferences; replaced with videoplay list from early Spark Summits [DOC] removed deadlinks from the events/conferences; replaced with videoplay list from early Spark Summits Oct 18, 2025
dmatrix and others added 3 commits October 19, 2025 19:36
site/index.html Outdated
<meta name="twitter:title" content=" Apache Spark&trade; - Unified Engine for large-scale data analytics">
<meta name="twitter:description" content="Apache Spark is a multi-language engine for executing data engineering, data science, and machine learning on single-node machines or clusters.">
<meta name="twitter:image" content="https://spark.apache.org/images/spark-twitter-card-large.jpg">
<meta name="twitter:image" content="http://localhost:4000/images/spark-twitter-card-large.jpg">
Copy link
Member

Choose a reason for hiding this comment

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

These changes are now making the build fail. I'd honestly just manually edit the change at this point to fix all the differences it complains about, rather than debug further. I can't merge this while the PR builder does not pass.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Blimey. I never touched any of the files it complains about. Well, I'll fix it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This build process seems like a slippery slope. I did not touch any of the pages it complains about, yet it reveals a new one after each triggered workflow. Do we keep doing that until it ends, or am I glaringly missing something here?

Copy link
Contributor Author

@dmatrix dmatrix Oct 23, 2025

Choose a reason for hiding this comment

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

What do you reckon we should do if this keeps spewing the next build issue?

Copy link
Member

Choose a reason for hiding this comment

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

I don't know, I haven't worked on this in a long time. I recall it worked but you had to use the ruby env in this repo and you need to Jekyll build and not serve

It isn't worth more work here unless you feel like debugging. Just get it to match by hand in this case.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did change by hand. See the last two committs. So something is off.
Can you lift the suggested changes, approve it, and I'll work with @HyukjinKwon to sort it out since we are cutting a 4.1 preview3 release soon, and it would be nice if it can be merge there.

Copy link
Member

Choose a reason for hiding this comment

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

I can still see "localhost:4000" in your change. Did you push?
You can see the failing check below and click through to see the issue.
No, the fastest thing here is to just get the changes in your PR here.

Copy link
Contributor Author

@dmatrix dmatrix Oct 25, 2025

Choose a reason for hiding this comment

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

Okay. I removed all references localhost in the actual site/.html files that the build was barfing on, committed and pushed. If the build fails, then I throw my towel in the ring .. ... lol

Copy link
Member

@srowen srowen left a comment

Choose a reason for hiding this comment

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

@HyukjinKwon can you merge? My local setup is not working anymore

@HyukjinKwon
Copy link
Member

Merged to asf-site.

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