[GitHub] ueshin commented on a change in pull request #163: Announce the schedule of 2019 Spark+AI summit at SF

classic Classic list List threaded Threaded
7 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] ueshin commented on a change in pull request #163: Announce the schedule of 2019 Spark+AI summit at SF

GitBox
ueshin commented on a change in pull request #163: Announce the schedule of 2019 Spark+AI summit at SF
URL: https://github.com/apache/spark-website/pull/163#discussion_r243130975
 
 

 ##########
 File path: site/sitemap.xml
 ##########
 @@ -139,657 +139,661 @@
 </url>
 <!-- Auto-generate sitemap for rest of site content -->
 <url>
-  <loc>https://spark.apache.org/releases/spark-release-2-4-0.html</loc>
+  <loc>http://localhost:4000/news/spark-ai-summit-apr-2019-agenda-posted.html</loc>
 
 Review comment:
   Still remaining `localhost:4000` in this file.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Noisy spark-website notifications

Nicholas Chammas
Can we somehow disable these new email alerts coming through for the Spark website repo?

On Wed, Dec 19, 2018 at 8:25 PM GitBox <[hidden email]> wrote:
ueshin commented on a change in pull request #163: Announce the schedule of 2019 Spark+AI summit at SF
URL: https://github.com/apache/spark-website/pull/163#discussion_r243130975



 ##########
 File path: site/sitemap.xml
 ##########
 @@ -139,657 +139,661 @@
 </url>
 <!-- Auto-generate sitemap for rest of site content -->
 <url>
-  <loc>https://spark.apache.org/releases/spark-release-2-4-0.html</loc>
+  <loc>http://localhost:4000/news/spark-ai-summit-apr-2019-agenda-posted.html</loc>

 Review comment:
   Still remaining `localhost:4000` in this file.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: Noisy spark-website notifications

rxin
I think there is an infra ticket open for it right now. 

On Wed, Dec 19, 2018 at 6:58 PM Nicholas Chammas <[hidden email]> wrote:
Can we somehow disable these new email alerts coming through for the Spark website repo?

On Wed, Dec 19, 2018 at 8:25 PM GitBox <[hidden email]> wrote:
ueshin commented on a change in pull request #163: Announce the schedule of 2019 Spark+AI summit at SF
URL: https://github.com/apache/spark-website/pull/163#discussion_r243130975



 ##########
 File path: site/sitemap.xml
 ##########
 @@ -139,657 +139,661 @@
 </url>
 <!-- Auto-generate sitemap for rest of site content -->
 <url>
-  <loc>https://spark.apache.org/releases/spark-release-2-4-0.html</loc>
+  <loc>http://localhost:4000/news/spark-ai-summit-apr-2019-agenda-posted.html</loc>

 Review comment:
   Still remaining `localhost:4000` in this file.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: Noisy spark-website notifications

cloud0fan
In reply to this post by Nicholas Chammas
+1, at least it should only send one email when a PR is merged.

On Thu, Dec 20, 2018 at 10:58 AM Nicholas Chammas <[hidden email]> wrote:
Can we somehow disable these new email alerts coming through for the Spark website repo?

On Wed, Dec 19, 2018 at 8:25 PM GitBox <[hidden email]> wrote:
ueshin commented on a change in pull request #163: Announce the schedule of 2019 Spark+AI summit at SF
URL: https://github.com/apache/spark-website/pull/163#discussion_r243130975



 ##########
 File path: site/sitemap.xml
 ##########
 @@ -139,657 +139,661 @@
 </url>
 <!-- Auto-generate sitemap for rest of site content -->
 <url>
-  <loc>https://spark.apache.org/releases/spark-release-2-4-0.html</loc>
+  <loc>http://localhost:4000/news/spark-ai-summit-apr-2019-agenda-posted.html</loc>

 Review comment:
   Still remaining `localhost:4000` in this file.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: Noisy spark-website notifications

Nicholas Chammas
I'd prefer it if we disabled all git notifications for spark-website. Folks who want to stay on top of what's happening with the site can simply watch the repo on GitHub, no?

On Wed, Dec 19, 2018 at 10:00 PM Wenchen Fan <[hidden email]> wrote:
+1, at least it should only send one email when a PR is merged.

On Thu, Dec 20, 2018 at 10:58 AM Nicholas Chammas <[hidden email]> wrote:
Can we somehow disable these new email alerts coming through for the Spark website repo?

On Wed, Dec 19, 2018 at 8:25 PM GitBox <[hidden email]> wrote:
ueshin commented on a change in pull request #163: Announce the schedule of 2019 Spark+AI summit at SF
URL: https://github.com/apache/spark-website/pull/163#discussion_r243130975



 ##########
 File path: site/sitemap.xml
 ##########
 @@ -139,657 +139,661 @@
 </url>
 <!-- Auto-generate sitemap for rest of site content -->
 <url>
-  <loc>https://spark.apache.org/releases/spark-release-2-4-0.html</loc>
+  <loc>http://localhost:4000/news/spark-ai-summit-apr-2019-agenda-posted.html</loc>

 Review comment:
   Still remaining `localhost:4000` in this file.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: Noisy spark-website notifications

Hyukjin Kwon
Yea, that's a bit noisy .. I would just completely disable it to be honest. I failed https://issues.apache.org/jira/browse/INFRA-17469 before. I would appreciate if there would be more inputs there :-)

2018년 12월 20일 (목) 오전 11:22, Nicholas Chammas <[hidden email]>님이 작성:
I'd prefer it if we disabled all git notifications for spark-website. Folks who want to stay on top of what's happening with the site can simply watch the repo on GitHub, no?

On Wed, Dec 19, 2018 at 10:00 PM Wenchen Fan <[hidden email]> wrote:
+1, at least it should only send one email when a PR is merged.

On Thu, Dec 20, 2018 at 10:58 AM Nicholas Chammas <[hidden email]> wrote:
Can we somehow disable these new email alerts coming through for the Spark website repo?

On Wed, Dec 19, 2018 at 8:25 PM GitBox <[hidden email]> wrote:
ueshin commented on a change in pull request #163: Announce the schedule of 2019 Spark+AI summit at SF
URL: https://github.com/apache/spark-website/pull/163#discussion_r243130975



 ##########
 File path: site/sitemap.xml
 ##########
 @@ -139,657 +139,661 @@
 </url>
 <!-- Auto-generate sitemap for rest of site content -->
 <url>
-  <loc>https://spark.apache.org/releases/spark-release-2-4-0.html</loc>
+  <loc>http://localhost:4000/news/spark-ai-summit-apr-2019-agenda-posted.html</loc>

 Review comment:
   Still remaining `localhost:4000` in this file.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

Re: Noisy spark-website notifications

rxin
I added my comment there too!


On Wed, Dec 19, 2018 at 7:26 PM, Hyukjin Kwon <[hidden email]> wrote:
Yea, that's a bit noisy .. I would just completely disable it to be honest. I failed https://issues.apache.org/jira/browse/INFRA-17469 before. I would appreciate if there would be more inputs there :-)

2018년 12월 20일 (목) 오전 11:22, Nicholas Chammas <[hidden email]>님이 작성:
I'd prefer it if we disabled all git notifications for spark-website. Folks who want to stay on top of what's happening with the site can simply watch the repo on GitHub, no?

On Wed, Dec 19, 2018 at 10:00 PM Wenchen Fan <[hidden email]> wrote:
+1, at least it should only send one email when a PR is merged.

On Thu, Dec 20, 2018 at 10:58 AM Nicholas Chammas <[hidden email]> wrote:
Can we somehow disable these new email alerts coming through for the Spark website repo?

On Wed, Dec 19, 2018 at 8:25 PM GitBox <[hidden email]> wrote:
ueshin commented on a change in pull request #163: Announce the schedule of 2019 Spark+AI summit at SF
URL: https://github.com/apache/spark-website/pull/163#discussion_r243130975



 ##########
 File path: site/sitemap.xml
 ##########
 @@ -139,657 +139,661 @@
 </url>
 <!-- Auto-generate sitemap for rest of site content -->
 <url>
-  <loc>https://spark.apache.org/releases/spark-release-2-4-0.html</loc>
+  <loc>http://localhost:4000/news/spark-ai-summit-apr-2019-agenda-posted.html</loc>

 Review comment:
   Still remaining `localhost:4000` in this file.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[hidden email]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe e-mail: [hidden email]