Implicit inflight client initialization about wing HOT 9 CLOSED

This might even tie into the "operational style sheets" discussion, where you can customize client initialization based on rules which aren't directly related to your biz-logic but rather to ops issues.

from wing.

Good call. Let's start with disallowing arguments for ~init() for now and see how it plays out.

(My suspicion is that the inflight client should have all the info it needs from preflight but i may be wrong)

@3p3r add to spec?

from wing.

Sounds good. This is an acceptable trade-off I think. The finalizer has the same issue. I'll enforce a compile error for these two methods.

from wing.

Related to #493

from wing.

Hi,

This issue hasn't seen activity in 60 days. Therefore, we are marking this issue as stale for now. It will be closed after 7 days.
Feel free to re-open this issue when there's an update or relevant information to be added.
Thanks!

from wing.

@yoav-steinberg , I think that the TLDR here is the inflight init is a no param function.

from wing.

I wonder how Wing should handle the case of performing async logic in inflight constructors, given our model that inflight is async by default. For example, in the following code

resource MyResource {
  b: cloud.Bucket;
  init(b: cloud.Bucket) {
    this.b = b;
  }
  inflight init() {
    this.b.put("data.json", "{}");
  }
  inflight do_stuff(
    let data = this.b.get("data.json");
    // do stuff
  ) 
}

Maybe the inflight client could look something like

class MyResourceClient {
  init({ b }) {
    this.$init_promises = [];
    this.b = b;
    $init_promises.push(this.b.put("data.json", "{}"));
  }
  async do_stuff() {
    await Promise.all(this.$init_promises);
    const data = (await this.b.get("data.json"));
    // do stuff
  }
}

Alternatively we could inherit the constraint that constructors are sync from the JS world, and give a compiler error (tell the user that this.b.put(...) needs to be prefixed with defer.

from wing.

@staycoolcall911 , moved this to P1 and Full TaskList

from wing.

Congrats! 🚀 This was released in Wing 0.10.3.

from wing.