adam-christian-software commented on code in PR #2599:
URL: https://github.com/apache/polaris/pull/2599#discussion_r2359463019


##########
README.md:
##########
@@ -45,20 +45,56 @@ Click 
[here](https://polaris.apache.org/in-dev/unreleased/getting-started/instal
 ## Project Structure
 
 Apache Polaris is organized into the following modules:
-
-- [`polaris-core`](./polaris-core/README.md) - The main Polaris entity 
definitions and core business logic
-- API modules (implementing the Iceberg REST API and Polaris management API):
-  - `polaris-api-management-model` - The Polaris management model
-  - `polaris-api-management-service` - The Polaris management service
-  - `polaris-api-iceberg-service` - The Iceberg REST service
-- Runtime modules:
-  - `polaris-runtime-service` - The runtime components of the Polaris server
-  - `polaris-runtime-defaults` - The runtime configuration defaults
-  - `polaris-server` - The Polaris server
-  - `polaris-admin` - The Polaris admin & maintenance tool
-- Persistence modules:
-  - `polaris-eclipselink` - The Eclipselink implementation of the 
MetaStoreManager interface
-  - `polaris-relational-jdbc` - The JDBC implementation of BasePersistence to 
be used via AtomicMetaStoreManager
+- Primary modules:
+  - [`polaris-core`](./polaris-core/README.md) - The main Polaris entity 
definitions and core business logic
+  - [API modules](./api/README.md) - Implements the Iceberg REST API and 
Polaris Management APIs:
+  - `polaris-api-management-model` - Polaris Management API model classes
+  - `polaris-api-management-service` - Polaris Management API service classes
+  - `polaris-api-iceberg-service` - The Iceberg REST service classes
+  - `polaris-api-catalog-service` - The Polaris Catalog API service classes
+  - Runtime modules:
+      - [`polaris-admin`](./runtime/admin/README.md) - The Polaris Admin Tool
+      - `polaris-runtime-common` - A package holding 
QuarkusRelationalJdbcConfiguration

Review Comment:
   Yeah, I agree that it would be nice to fold into a different module. I'll 
remove it from the list for right now though. :)



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