You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@geode.apache.org by Dan Smith <da...@vmware.com> on 2020/09/28 19:21:18 UTC
[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://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&v=4]<https://github.com/apache/geode/pull/5566>
GEODE-8522: Switching exception log back to debug (merge to 1.13) by upthewaterspout · Pull Request #5566 · apache/geode<https://github.com/apache/geode/pull/5566>
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
Posted by Joris Melchior <jm...@vmware.com>.
+1
On 2020-09-28, 3:21 PM, "Dan Smith" <da...@vmware.com> 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%2F5566&data=02%7C01%7Cjmelchior%40vmware.com%7Cff464688426a4c9a7e6808d863e3b333%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176923816350&sdata=mPB6w8RD0e44sLYEgcJCoP%2Fu%2BPepilHevImzjmjfkmA%3D&reserved=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%3D4&data=02%7C01%7Cjmelchior%40vmware.com%7Cff464688426a4c9a7e6808d863e3b333%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176923816350&sdata=IBtTHJOQigbGaDdCWnWogLjjZsHFgEzNDzde5CW7TDQ%3D&reserved=0]<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566&data=02%7C01%7Cjmelchior%40vmware.com%7Cff464688426a4c9a7e6808d863e3b333%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176923816350&sdata=mPB6w8RD0e44sLYEgcJCoP%2Fu%2BPepilHevImzjmjfkmA%3D&reserved=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%2F5566&data=02%7C01%7Cjmelchior%40vmware.com%7Cff464688426a4c9a7e6808d863e3b333%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176923816350&sdata=mPB6w8RD0e44sLYEgcJCoP%2Fu%2BPepilHevImzjmjfkmA%3D&reserved=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
Posted by Dan Smith <da...@vmware.com>.
Thanks all, I merged the change.
-Dan
________________________________
From: Raymond Ingles <ri...@vmware.com>
Sent: Tuesday, September 29, 2020 7:56 AM
To: dev@geode.apache.org <de...@geode.apache.org>
Subject: Re: [DISCUSS] One more 1.13 change
+1
On 9/28/20, 3:21 PM, "Dan Smith" <da...@vmware.com> 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%2F5566&data=02%7C01%7Cdasmith%40vmware.com%7C351d4f5919a6404bccec08d8648bc30e%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369898722059642&sdata=HNA%2FwPOsdg2eEwQrQcJgektNwJKQdHpowa%2BxOzMf37U%3D&reserved=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%3D4&data=02%7C01%7Cdasmith%40vmware.com%7C351d4f5919a6404bccec08d8648bc30e%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369898722059642&sdata=n95PeiYayrtWKBh418Z0yI4X6hJZQ%2BZ4MZGvvKMBg00%3D&reserved=0]<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566&data=02%7C01%7Cdasmith%40vmware.com%7C351d4f5919a6404bccec08d8648bc30e%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369898722059642&sdata=HNA%2FwPOsdg2eEwQrQcJgektNwJKQdHpowa%2BxOzMf37U%3D&reserved=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%2F5566&data=02%7C01%7Cdasmith%40vmware.com%7C351d4f5919a6404bccec08d8648bc30e%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369898722059642&sdata=HNA%2FwPOsdg2eEwQrQcJgektNwJKQdHpowa%2BxOzMf37U%3D&reserved=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
Posted by Raymond Ingles <ri...@vmware.com>.
+1
On 9/28/20, 3:21 PM, "Dan Smith" <da...@vmware.com> 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%2F5566&data=02%7C01%7Cringles%40vmware.com%7C7333b0ccfcef446143e308d863e3b629%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176949268879&sdata=yG0RpDG9d9LCdgQauByTV%2Bfh7nHBWL8rvyX6ZZRKXSE%3D&reserved=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%3D4&data=02%7C01%7Cringles%40vmware.com%7C7333b0ccfcef446143e308d863e3b629%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176949268879&sdata=z%2Fa23PMHASn3GEALwPqI7AXr4o3om1bpz0OkRpyn3rw%3D&reserved=0]<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566&data=02%7C01%7Cringles%40vmware.com%7C7333b0ccfcef446143e308d863e3b629%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176949268879&sdata=yG0RpDG9d9LCdgQauByTV%2Bfh7nHBWL8rvyX6ZZRKXSE%3D&reserved=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%2F5566&data=02%7C01%7Cringles%40vmware.com%7C7333b0ccfcef446143e308d863e3b629%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176949268879&sdata=yG0RpDG9d9LCdgQauByTV%2Bfh7nHBWL8rvyX6ZZRKXSE%3D&reserved=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
Posted by John Blum <jb...@vmware.com>.
+1
________________________________
From: Dan Smith <da...@vmware.com>
Sent: Monday, September 28, 2020 12:21 PM
To: dev@geode.apache.org <de...@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%2F5566&data=02%7C01%7Cjblum%40vmware.com%7C6c125c591647400fdcd308d863e3b3bb%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176913751568&sdata=N7j85l7TN0l%2FarLxJkl1%2FtwBLSWrrMfTYVRzF8Xk12s%3D&reserved=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%3D4&data=02%7C01%7Cjblum%40vmware.com%7C6c125c591647400fdcd308d863e3b3bb%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176913761570&sdata=Q80xKsvdce1%2BIMXybp1%2BN2UnZjDkX77r9Kkrk3%2FK5os%3D&reserved=0]<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566&data=02%7C01%7Cjblum%40vmware.com%7C6c125c591647400fdcd308d863e3b3bb%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176913761570&sdata=SY6insLY4jwNd1XZibeXxbBTMQ%2FTofSWCriC5SKqZLI%3D&reserved=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%2F5566&data=02%7C01%7Cjblum%40vmware.com%7C6c125c591647400fdcd308d863e3b3bb%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176913761570&sdata=SY6insLY4jwNd1XZibeXxbBTMQ%2FTofSWCriC5SKqZLI%3D&reserved=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
Posted by Mario Kevo <ma...@est.tech>.
+1
________________________________
Šalje: Patrick Johnson <jp...@vmware.com>
Poslano: 28. rujna 2020. 21:27
Prima: dev@geode.apache.org <de...@geode.apache.org>
Predmet: Re: [DISCUSS] One more 1.13 change
+1
> On Sep 28, 2020, at 12:21 PM, Dan Smith <da...@vmware.com> 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%2F5566&data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210&sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3D&reserved=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%3D4&data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210&sdata=KNHUq8JyZOd5KSto0kgg6vJJNcZq3HLTN67uj2xzBg4%3D&reserved=0]<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566&data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210&sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3D&reserved=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%2F5566&data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210&sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3D&reserved=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
Posted by Patrick Johnson <jp...@vmware.com>.
+1
> On Sep 28, 2020, at 12:21 PM, Dan Smith <da...@vmware.com> 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%2F5566&data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210&sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3D&reserved=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%3D4&data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210&sdata=KNHUq8JyZOd5KSto0kgg6vJJNcZq3HLTN67uj2xzBg4%3D&reserved=0]<https://nam04.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fapache%2Fgeode%2Fpull%2F5566&data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210&sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3D&reserved=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%2F5566&data=02%7C01%7Cjpatrick%40vmware.com%7Cb32ecd430b404e610fe708d863e3b4a5%7Cb39138ca3cee4b4aa4d6cd83d9dd62f0%7C0%7C1%7C637369176927680210&sdata=lgNl51lgbcV1LmuihEAhkw8mwxbfAbUqawqsEgwdNUA%3D&reserved=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
>