Skip to content

Commit d8dd8a6

Browse files
authored
fix(cubesql): Failed printing to stdout: Resource temporarily unavailable (#6272)
The root cause is a non-blocking STDOUT set by Node.js at startup. I don't think it's a correct idea to set up O_NONBLOCK for STDOUT. Let's hide some trace logs as a workaround for this issue.
1 parent cd32266 commit d8dd8a6

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

packages/cubejs-backend-native/src/transport.rs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,10 @@ impl TransportService for NodeBridgeTransport {
8989
Some(extra),
9090
)
9191
.await?;
92+
#[cfg(debug_assertions)]
9293
trace!("[transport] Meta <- {:?}", response);
94+
#[cfg(not(debug_assertions))]
95+
trace!("[transport] Meta <- <hidden>", response);
9396

9497
Ok(Arc::new(MetaContext::new(
9598
response.cubes.unwrap_or_default(),
@@ -131,7 +134,10 @@ impl TransportService for NodeBridgeTransport {
131134
Some(extra),
132135
)
133136
.await?;
137+
#[cfg(debug_assertions)]
134138
trace!("[transport] Request <- {:?}", response);
139+
#[cfg(not(debug_assertions))]
140+
trace!("[transport] Request <- <hidden>", response);
135141

136142
let load_err = match serde_json::from_value::<V1LoadResponse>(response.clone()) {
137143
Ok(r) => {

0 commit comments

Comments
 (0)