Hi, and welcome to OpenStreetMap! Since you requested a review, I looked over your changes here. Looks great over all! Here are a couple of suggestions/issues.
- The name tag is only for the common or official name of something, and shouldn't be used to describe a feature. In this case, the fact that this is a pavilion structure should be represented by other tags.
- Looks like you set layer=-1 to get rid of the warning about it crossing the foot path. Since it's a roof structure that goes over, it should be layer=1.
Overall, I suggest the following tags for this:

amenity=shelter
building=roof
shelter_type=picnic_shelter
layer=1

Let me know if you have any questions, and thanks for your contribution!