Re: [DISCUSS] One more 1.13 change

2020-09-29 Thread Dan Smith 
Thanks all, I merged the change.

-Dan

From: Raymond Ingles 
Sent: Tuesday, September 29, 2020 7:56 AM
To: dev@geode.apache.org 
Subject: Re: [DISCUSS] One more 1.13 change

+1

On 9/28/20, 3:21 PM, "Dan Smith"  wrote:

Hi,

I'd like to backport this change to support/1.13 as well

GEODE-8522: Switching exception log back to debug - 
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cdasmith%40vmware.com%7C351d4f5919a6404bccec08d8648bc30e%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369898722059642sdata=HNA%2FwPOsdg2eEwQrQcJgektNwJKQdHpowa%2BxOzMf37U%3Dreserved=0

This cleans up some noise in our logs that customers might see.

[https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Favatars3.githubusercontent.com%2Fu%2F47359%3Fs%3D400%26v%3D4data=02%7C01%7Cdasmith%40vmware.com%7C351d4f5919a6404bccec08d8648bc30e%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369898722059642sdata=n95PeiYayrtWKBh418Z0yI4X6hJZQ%2BZ4MZGvvKMBg00%3Dreserved=0]<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cdasmith%40vmware.com%7C351d4f5919a6404bccec08d8648bc30e%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369898722059642sdata=HNA%2FwPOsdg2eEwQrQcJgektNwJKQdHpowa%2BxOzMf37U%3Dreserved=0>
GEODE-8522: Switching exception log back to debug (merge to 1.13) by 
upthewaterspout · Pull Request #5566 · 
apache/geode<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cdasmith%40vmware.com%7C351d4f5919a6404bccec08d8648bc30e%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369898722059642sdata=HNA%2FwPOsdg2eEwQrQcJgektNwJKQdHpowa%2BxOzMf37U%3Dreserved=0>
This log message happens during the course of normal startup of multiple 
locators. We should not be logging a full stack trace during normal startup. 
(cherry picked from commit 3df057c) Thank you f...
github.com

Re: [DISCUSS] One more 1.13 change

2020-09-29 Thread Raymond Ingles 
+1

On 9/28/20, 3:21 PM, "Dan Smith"  wrote:

Hi,

I'd like to backport this change to support/1.13 as well

GEODE-8522: Switching exception log back to debug - 
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cringles%40vmware.com%7C7333b0ccfcef446143e308d863e3b629%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176949268879sdata=yG0RpDG9d9LCdgQauByTV%2Bfh7nHBWL8rvyX6ZZRKXSE%3Dreserved=0

This cleans up some noise in our logs that customers might see.

[https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Favatars3.githubusercontent.com%2Fu%2F47359%3Fs%3D400%26v%3D4data=02%7C01%7Cringles%40vmware.com%7C7333b0ccfcef446143e308d863e3b629%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176949268879sdata=z%2Fa23PMHASn3GEALwPqI7AXr4o3om1bpz0OkRpyn3rw%3Dreserved=0]
GEODE-8522: Switching exception log back to debug (merge to 1.13) by 
upthewaterspout · Pull Request #5566 · 
apache/geode
This log message happens during the course of normal startup of multiple 
locators. We should not be logging a full stack trace during normal startup. 
(cherry picked from commit 3df057c) Thank you f...
github.com

Re: [DISCUSS] One more 1.13 change

2020-09-28 Thread Joris Melchior 
+1

On 2020-09-28, 3:21 PM, "Dan Smith"  wrote:

Hi,

I'd like to backport this change to support/1.13 as well

GEODE-8522: Switching exception log back to debug - 
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cjmelchior%40vmware.com%7Cff464688426a4c9a7e6808d863e3b333%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176923816350sdata=mPB6w8RD0e44sLYEgcJCoP%2Fu%2BPepilHevImzjmjfkmA%3Dreserved=0

This cleans up some noise in our logs that customers might see.

[https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Favatars3.githubusercontent.com%2Fu%2F47359%3Fs%3D400%26v%3D4data=02%7C01%7Cjmelchior%40vmware.com%7Cff464688426a4c9a7e6808d863e3b333%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176923816350sdata=IBtTHJOQigbGaDdCWnWogLjjZsHFgEzNDzde5CW7TDQ%3Dreserved=0]
GEODE-8522: Switching exception log back to debug (merge to 1.13) by 
upthewaterspout · Pull Request #5566 · 
apache/geode
This log message happens during the course of normal startup of multiple 
locators. We should not be logging a full stack trace during normal startup. 
(cherry picked from commit 3df057c) Thank you f...
github.com

Re: [DISCUSS] One more 1.13 change

2020-09-28 Thread Mario Kevo 
+1

Šalje: Patrick Johnson 
Poslano: 28. rujna 2020. 21:27
Prima: dev@geode.apache.org 
Predmet: Re: [DISCUSS] One more 1.13 change

+1

> On Sep 28, 2020, at 12:21 PM, Dan Smith  wrote:
>
> Hi,
>
> I'd like to backport this change to support/1.13 as well
>
> GEODE-8522: Switching exception log back to debug - 
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3Dreserved=0
>
> This cleans up some noise in our logs that customers might see.
> [https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Favatars3.githubusercontent.com%2Fu%2F47359%3Fs%3D400%26v%3D4data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210sdata=KNHUq8JyZOd5KSto0kgg6vJJNcZq3HLTN67uj2xzBg4%3Dreserved=0]<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3Dreserved=0>
> GEODE-8522: Switching exception log back to debug (merge to 1.13) by 
> upthewaterspout · Pull Request #5566 · 
> apache/geode<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3Dreserved=0>
> This log message happens during the course of normal startup of multiple 
> locators. We should not be logging a full stack trace during normal startup. 
> (cherry picked from commit 3df057c) Thank you f...
> github.com
>

Re: [DISCUSS] One more 1.13 change

2020-09-28 Thread Patrick Johnson 
+1

> On Sep 28, 2020, at 12:21 PM, Dan Smith  wrote:
> 
> Hi,
> 
> I'd like to backport this change to support/1.13 as well
> 
> GEODE-8522: Switching exception log back to debug - 
> https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3Dreserved=0
> 
> This cleans up some noise in our logs that customers might see.
> [https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Favatars3.githubusercontent.com%2Fu%2F47359%3Fs%3D400%26v%3D4data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210sdata=KNHUq8JyZOd5KSto0kgg6vJJNcZq3HLTN67uj2xzBg4%3Dreserved=0]
> GEODE-8522: Switching exception log back to debug (merge to 1.13) by 
> upthewaterspout · Pull Request #5566 · 
> apache/geode
> This log message happens during the course of normal startup of multiple 
> locators. We should not be logging a full stack trace during normal startup. 
> (cherry picked from commit 3df057c) Thank you f...
> github.com
>

Re: [DISCUSS] One more 1.13 change

2020-09-28 Thread John Blum 
+1


From: Dan Smith 
Sent: Monday, September 28, 2020 12:21 PM
To: dev@geode.apache.org 
Subject: [DISCUSS] One more 1.13 change

Hi,

I'd like to backport this change to support/1.13 as well

GEODE-8522: Switching exception log back to debug - 
https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cjblum%40vmware.com%7C6c125c591647400fdcd308d863e3b3bb%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176913751568sdata=N7j85l7TN0l%2FarLxJkl1%2FtwBLSWrrMfTYVRzF8Xk12s%3Dreserved=0

This cleans up some noise in our logs that customers might see.
[https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Favatars3.githubusercontent.com%2Fu%2F47359%3Fs%3D400%26v%3D4data=02%7C01%7Cjblum%40vmware.com%7C6c125c591647400fdcd308d863e3b3bb%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176913761570sdata=Q80xKsvdce1%2BIMXybp1%2BN2UnZjDkX77r9Kkrk3%2FK5os%3Dreserved=0]<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cjblum%40vmware.com%7C6c125c591647400fdcd308d863e3b3bb%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176913761570sdata=SY6insLY4jwNd1XZibeXxbBTMQ%2FTofSWCriC5SKqZLI%3Dreserved=0>
GEODE-8522: Switching exception log back to debug (merge to 1.13) by 
upthewaterspout · Pull Request #5566 · 
apache/geode<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566data=02%7C01%7Cjblum%40vmware.com%7C6c125c591647400fdcd308d863e3b3bb%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176913761570sdata=SY6insLY4jwNd1XZibeXxbBTMQ%2FTofSWCriC5SKqZLI%3Dreserved=0>
This log message happens during the course of normal startup of multiple 
locators. We should not be logging a full stack trace during normal startup. 
(cherry picked from commit 3df057c) Thank you f...
github.com

[DISCUSS] One more 1.13 change

2020-09-28 Thread Dan Smith 
Hi,

I'd like to backport this change to support/1.13 as well

GEODE-8522: Switching exception log back to debug - 
https://github.com/apache/geode/pull/5566

This cleans up some noise in our logs that customers might see.
[https://avatars3.githubusercontent.com/u/47359?s=400=4]
GEODE-8522: Switching exception log back to debug (merge to 1.13) by 
upthewaterspout · Pull Request #5566 · 
apache/geode
This log message happens during the course of normal startup of multiple 
locators. We should not be logging a full stack trace during normal startup. 
(cherry picked from commit 3df057c) Thank you f...
github.com