yasithdev opened a new pull request, #601:
URL: https://github.com/apache/airavata/pull/601
## Summary
- **Module flattening**: Moved
`modules/{research-service,agent-service,file-server}` under `airavata-api/` as
sub-modules. Deleted `modules/` directory entirely.
- **Server collapse**: Collapsed `airavata-rest-server`,
`airavata-grpc-server`, `airavata-thrift-server` into
`airavata-server/{rest,grpc,thrift}` sub-modules with package renames to
`org.apache.airavata.server.{rest,grpc,thrift}.*`
- **Internal Thrift client elimination**: Replaced ~29 internal Thrift
client connection points with direct Spring bean injection. No more network
round-trips within the JVM.
- Deleted `SharingRegistryServiceClientFactory`,
`CredentialStoreClientFactory`
- Replaced `ThriftClientPool<RegistryService.Client>` in 12+
scheduler/orchestrator/monitor classes with direct `RegistryServerHandler`
injection
- Replaced `SharingRegistryService.Client` in 4 classes with direct
`SharingRegistryServerHandler`
- Replaced `CredentialStoreService.Client` in 5 classes with direct
`CredentialStoreServerHandler`
- Replaced `UserProfileService.Client` in research-service with direct
`UserProfileRepository`
- Replaced `Airavata.Client` in agent-service with direct `Airavata.Iface`
handler
- Replaced `ThriftClientPool` in file-server with direct
`RegistryService.Iface` handler
### Final structure
```
airavata/
├── airavata-api/ (core lib)
│ ├── research-service/ (sub-module)
│ ├── agent-service/ (sub-module)
│ └── file-server/ (sub-module)
├── airavata-server/ (unified server)
│ ├── rest/ (sub-module)
│ ├── grpc/ (sub-module)
│ └── thrift/ (sub-module)
└── integration-tests/
```
**445 files changed, -25,071 lines net**
## Test Plan
- [x] Full `mvn clean install -DskipTests -T4` passes (all 10 modules)
- [x] Zero internal `ThriftClientPool`/`ClientFactory` usages remain in app
code
- [ ] `tilt up` starts unified server
- [ ] REST, gRPC, Thrift endpoints all accessible
- [ ] Internal service calls work without Thrift serialization overhead
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
