yasithdev opened a new pull request, #643:
URL: https://github.com/apache/airavata/pull/643
A module-isolation fix. ResourceEntity (table RESOURCE), TagEntity,
ResourceStarEntity, their three repositories, and the four research-only enums
(ResourceTypeEnum / PrivacyEnum / StateEnum / StatusEnum) physically lived in
sharing-service but are used **only** by research-service — which was reaching
into sharing's internal model/repository packages 28 times (the single largest
cross-module boundary leak in airavata-api). This relocates them to
org.apache.airavata.research.{model,repository}. It is a **package-only move
with no schema change**: the RESOURCE / TAG / RESOURCE_STAR tables and their
Flyway migrations are untouched (those tables are now conceptually owned by
research-service, served by the same SQL), and the broad @EntityScan over
org.apache.airavata still discovers the entities at their new package. All
consumers (ResearchResourceService, ResearchProjectService,
ResearchResourceGrpcHandler, DevDataInitializer, DatasetResourceEntity,
RepositoryResourceEntity, Resea
rchProjectEntity, ResearchProjectRepository) and the shared test persistence
unit are repointed. Result: research-service's imports of
sharing.model/repository drop from 28 to 0. Behavior-preserving; the full
reactor builds green and research-service's 53 unit tests pass (the PU
bootstrap is exercised by the integration suite in CI).
--
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]
