[PR] Fix hasListeners() always returning true when no event listeners configured [polaris]

[via GitHub]

hyongtao-code opened a new pull request, #4493:
URL: https://github.com/apache/polaris/pull/4493

   Long log:
   
    - Change `enabledEventTypes` initialization from `EnumSet.allOf()` to 
`EnumSet.noneOf()` to fix incorrect `hasListeners()` behavior
    - Add backward compatibility: when no listeners are explicitly configured, 
enable all event types
   
    BTW, the original code won’t cause any bugs, but it will make the 
`PolarisEventListeners` class lose its configurability.
   
   Signed-off-by : Yongtao Huang <yongtaoh2...@gmail.com>
   
   <!--
   📝 Describe what changes you're proposing, especially breaking or user-facing 
changes. 
   📖 See https://github.com/apache/polaris/blob/main/CONTRIBUTING.md for more.
   -->
   
   ## Checklist
   - [ ] 🛡️ Don't disclose security issues! (contact secur...@apache.org)
   - [ ] 🔗 Clearly explained why the changes are needed, or linked related 
issues: Fixes #
   - [ ] 🧪 Added/updated tests with good coverage, or manually tested (and 
explained how)
   - [ ] 💡 Added comments for complex logic
   - [ ] 🧾 Updated `CHANGELOG.md` (if needed)
   - [ ] 📚 Updated documentation in `site/content/in-dev/unreleased` (if needed)
   


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

To unsubscribe, e-mail: issues-unsubscr...@polaris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org