[I] [Compiler][Go] date fields emit an unused time import; gRPC stubs emit invalid selectors for nested RPC payload types [fory]

["ayush00git (via GitHub)" <gi...@apache.org>]

ayush00git opened a new issue, #4154:
URL: https://github.com/apache/fory/issues/4154

   Two Go codegen bugs in the compiler:
   
   1. A schema whose only temporal field is \`date\` adds \`import "time"\` to the generated file, but \`date\` maps to \`fory.Date\`, which never references the \`time\` package — the generated Go fails to compile with \`"time" imported and not used\`.
   
   Repro:
   ```
   message M { date d = 1; }
   ```
   
   2. The gRPC service generator resolves nested payload types to dot-qualified names (\`*Local.Deep\`, \`*shared.Outer.Inner\`), but the model generator flattens nested types (\`Local_Deep\`), so the generated \`_grpc.go\` fails to compile on an invalid selector.
   
   Repro:
   ```
   rpc DeepEcho (Local.Deep) returns (Outer.Inner);
   ```
   
   Fixes with regression tests are prepared; PR to follow.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@fory.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@fory.apache.org
For additional commands, e-mail: commits-help@fory.apache.org