yasithdev opened a new pull request, #647:
URL: https://github.com/apache/airavata/pull/647

   Now that the sharing code lives in `iam-service`, this collapses its 
`org.apache.airavata.sharing.{config,event,grpc,model,repository,service,util}` 
Java packages into the matching `org.apache.airavata.iam.*` packages so the 
module has a single coherent namespace. It is an internal, server-side-only 
move: the proto packages (`org.apache.airavata.api.sharing` / `.groupmanager`) 
and the shared `org.apache.airavata.sharing.registry.models.proto` types are 
deliberately left untouched, and every external reference to sharing went 
through those proto/facade types, so there is **no consumer import or gRPC/REST 
wire-contract change**. The redundant `SharingServiceConfig` is removed because 
`IamServiceConfig` already component- and entity-scans 
`org.apache.airavata.iam`, which now covers the moved beans and entities. The 
one name clash — two different `AbstractRepository` base classes — is resolved 
by renaming the entity-direct sharing base to `AbstractSharingRepository`; it 
has a diff
 erent type arity and a checked-exception contract than iam's model-converting 
`AbstractRepository`, so the two are kept distinct rather than force-merged. 
The full reactor builds green and iam-service tests (including the relocated 
`ResourceSharingServiceTest`) pass.
   
   A follow-up PR will nest the proto packages under `api.iam.*` 
(wire-breaking; includes Python SDK regeneration).


-- 
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]

Reply via email to