Merge development (with our current Datawarehouse code) into the AWS branch.
#232
Bugbot Review
Bugbot Analysis Progress (1m 18s elapsed)
✅ Gathered PR context (2s)
✅ Analyzed code changes (1s)
✅ Completed bug detection — 2 potential bugs found (1m 9s)
✅ Validation and filtering completed (0s)
✅ Posted analysis results — 2 bugs reported (5s)
✅ Analysis completed successfully (0s)
Final Result: Bugbot completed review and found 2 potential issues
Request ID: serverGenReqId_1c290194-1117-4187-84d0-41a37f872c9b
Details
Bug: Fixed: Correct Writable Property Name Used
Incorrect property check for stream writability. The code checks errorStream.Writable (capital W), but Node.js streams use the lowercase property writable (lowercase w) to check if a stream is writable. This check will always be falsy, causing the stream to never be properly closed. Should be errorStream.writable instead of errorStream.Writable.
common/datawarehouse/load.js#L317-L318
connectors/common/datawarehouse/load.js
Lines 317 to 318 in 1a4a4c8
