The GitHub Actions job "Build" on rocketmq-clients.git/master has failed. Run started by GitHub user RongtongJin (triggered by RongtongJin).
Head commit for run: d71c6c53acdd773f695575f75284d9c1cce3fdd9 / zhaohai <[email protected]> Fix priority issue.Supports lightweight messaging. (#1215) * fix: Add LITE and PRIORITY message types to match protocol definition - Add LITE = 5 and PRIORITY = 6 to MessageType enum in definition.proto - Align Node.js client with the latest protocol specification - Fix inconsistency between proto definition and generated TypeScript/JS files * fix: Update proto submodule and nodejs proto definition - Update protos submodule to include LITE and PRIORITY message types - Sync nodejs proto definition with protocol changes - Ensure npm run init preserves LITE and PRIORITY types * fix: Sync nodejs proto definition with updated protocol - Update definition.proto to include LITE and PRIORITY message types - Generated from updated protos submodule using npm run init - Ensures Node.js client stays in sync with protocol changes * feat: Add priority message support for Node.js client - Add PRIORITY message type detection in PublishingMessage - Set priority field in toProtobuf method - Validate mutual exclusivity: priority cannot be used with messageGroup, deliveryTimestamp - Ensure transactional messages cannot have priority set - Add ProducerPriorityMessageExample demonstrating priority usage - Align with Java implementation semantics Key changes: 1. PublishingMessage: Detect PRIORITY message type and set it accordingly 2. PublishingMessage.toProtobuf(): Include priority in SystemProperties 3. Message: Validate priority >= 0 and mutual exclusivity constraints 4. Example: Comprehensive demo showing priority levels and validation * fix: Use optional chaining for setPriority to avoid TS error - Use (systemProperties as any).setPriority?.() as workaround - Avoids TypeScript error when proto definition hasn't been updated yet - Will work correctly once official proto includes priority field - Maintains backward compatibility * update proto * update proto * add lite message * Change protos submodule URL to HTTPS for CI compatibility * rollback * Complete mutual exclusion verification * Support Lite Push * update protos version * add lite simple consumer * fix getClientType * fix getClientType * update startup * update name * fix producer getClientType * fix producer getClientType * update proto * All 12 review comments from reviewer zk-drizzle have been completed. * remove log * optimize simpleConsumer * remove delay * update example * update example * Securely handle concurrent refresh requests and exceptional scenarios * update producer delay * fix lite push * fix equals hashcode * Optimize hashCodeOfString * Optimize log * Optimization of timer cleaning sequence * TelemeterySession releases logic optimization Report URL: https://github.com/apache/rocketmq-clients/actions/runs/24826266057 With regards, GitHub Actions via GitBox
