The GitHub Actions job "Flink Connector HTTP CI" on flink-connector-http.git/feature/improve-error-logging has failed. Run started by GitHub user featzhang (triggered by featzhang).
Head commit for run: e361cf4268f3bd90e6d6717d9b7574d027856a72 / FeatZhang <[email protected]> refactor: integrate error logging into existing HttpLogger Per reviewer feedback, removed HttpErrorLogger class and integrated error logging functionality directly into the existing HttpLogger. Changes: - Extended HttpLogger with http.error.log.severity config (OFF/ERROR/WARN/INFO) - Enhanced logLookupError() and logSinkError() to respect error severity level - Maintained http.logging.level (MIN/REQ_RESP/MAX) for content verbosity - Two orthogonal dimensions: content detail × error severity - Backward compatible: default behavior unchanged (errors at DEBUG) - Production-friendly: set http.error.log.severity=ERROR for monitoring Removed: - HttpErrorLogger class and related files - Duplicate error logging configuration (ERROR_LOG_LEVEL, ERROR_LOG_INCLUDE_BODY, ERROR_LOG_BODY_MAX_SIZE) Kept: - Existing HttpLogger infrastructure - Sensitive header masking - Continue-on-error awareness (DEBUG when tolerated) The refactored design provides: 1. http.logging.level (MIN/REQ_RESP/MAX) controls content verbosity 2. http.error.log.severity (OFF/INFO/WARN/ERROR) controls log level for actual errors 3. Errors with continue-on-error=true always log at DEBUG (tolerated errors) 4. Errors with continue-on-error=false respect http.error.log.severity setting Report URL: https://github.com/apache/flink-connector-http/actions/runs/24790385621 With regards, GitHub Actions via GitBox
