daml icon indicating copy to clipboard operation
daml copied to clipboard

pass in interface ID to create-and-exercise, exercise-by-key commands

Open S11001001 opened this issue 3 years ago • 3 comments

Spin off #13920. When #13653 introduces the optional interface ID to these commands, change CreateAnd.java and ByKey.java to pass in the interface ID as mentioned in TODOs in that file.

Note https://github.com/digital-asset/daml/issues/13653#issuecomment-1142178491 for upstream status.

https://github.com/digital-asset/daml/blob/f8766e5ae0efade4c90fc53908dcddabdc6c2712/language-support/java/bindings/src/main/java/com/daml/ledger/javaapi/data/codegen/CreateAnd.java#L44-L46

https://github.com/digital-asset/daml/blob/f8766e5ae0efade4c90fc53908dcddabdc6c2712/language-support/java/bindings/src/main/java/com/daml/ledger/javaapi/data/codegen/ByKey.java#L43-L44

S11001001 avatar Jun 01 '22 17:06 S11001001

When https://github.com/digital-asset/daml/issues/13653 introduces the optional interface ID to these commands,

Does it mean that there will be an optional interface ID appearing in the command proto?

https://github.com/digital-asset/daml/blob/9d5115da0bad39a880d3a077948831f6c9e975cd/ledger-api/grpc-definitions/com/daml/ledger/api/v1/commands.proto

cc @ray-roestenburg-da

chunlokling-da avatar Aug 11 '22 14:08 chunlokling-da

@chunlokling-da yes I'm expecting a change like that. I think I've seen something in PRs but can't seem to find it right now. (Or maybe I'm confusing it with the changes to interface subscriptions)

ray-roestenburg-da avatar Aug 11 '22 15:08 ray-roestenburg-da

@S11001001 Is create-and-exercise with interface ID blocked, or just exercise-by-key?

ray-roestenburg-da avatar Sep 12 '22 08:09 ray-roestenburg-da

Blocked until a choice_interface_id field is added to the gRPC service, this was discussed with @remyhaemmerle-da in our Zurich meeting but no further work has been done on this.

ray-roestenburg-da avatar Dec 20 '22 16:12 ray-roestenburg-da

Hi Team,

I want to bring to your attention that an issue was recently raised in the support portal where a customer was unsure whether this missing functionality was a bug in our code.

Is there a plan to implement any sort of Interface Keys feature in an upcoming release?

Michael Godfrey DA Support

mgodf89 avatar Aug 25 '23 15:08 mgodf89

@michael-godfrey-da For what it's worth, this issue doesn't address notional interface keys; the key here is still a template key, which is still an unambiguous way to select the subject of an exercise-by-key when the choice is defined on an interface. The feature's absence is known.

Interface keys would be a quite more involved feature; I don't know whether they are in the pipeline. Given the problems with key uniqueness already, it wouldn't surprise me if they were found to be intractable.

S11001001 avatar Aug 28 '23 14:08 S11001001

This is a feature request. This needs to be prioritized by product.

remyhaemmerle-da avatar Aug 29 '23 17:08 remyhaemmerle-da