MonkeyCanCode commented on code in PR #2530:
URL: https://github.com/apache/polaris/pull/2530#discussion_r2335277682


##########
runtime/distribution/build.gradle.kts:
##########
@@ -71,6 +71,17 @@ distributions {
             into("bin") {
                 from("bin/server")
                 from("bin/admin")
+                from("${rootDir}/polaris")
+            }
+
+            into("bin/client") {
+                from("${rootDir}/client/python") {
+                    include("**/*")

Review Comment:
   Personally, I think we should only publish wheel file produced by 
https://github.com/apache/polaris/pull/2425. Reasoning is as 
`client/python/pyproject.toml` is invoking `generate_clients.py` which refs 
OpenAPI files outside `client/python`, people won't be able to build this 
anyway with only these files without restore the structure. Thus, I make more 
sense to me for publishing the wheel file instead which will have everything 
generated. You can do so via following:
   ```
   make client-build
   ```
   
   Once completed, you will then have a wheel file under `client/python/dist/` 
such as following:
   ```
   client/python/dist/polaris-1.0.0-cp313-cp313-macosx_15_0_arm64.whl
   ```
   
   The content of the wheel files will then be following:
   ```
   ➜  polaris git:(main) unzip -l 
client/python/dist/polaris-1.0.0-cp313-cp313-macosx_15_0_arm64.whl
   Archive:  client/python/dist/polaris-1.0.0-cp313-cp313-macosx_15_0_arm64.whl
     Length      Date    Time    Name
   ---------  ---------- -----   ----
         789  01-01-2016 00:00   cli/__init__.py
        9979  01-01-2016 00:00   cli/command/__init__.py
        5362  01-01-2016 00:00   cli/command/catalog_roles.py
       19082  01-01-2016 00:00   cli/command/catalogs.py
        4239  01-01-2016 00:00   cli/command/namespaces.py
        5328  01-01-2016 00:00   cli/command/principal_roles.py
        6954  01-01-2016 00:00   cli/command/principals.py
        6138  01-01-2016 00:00   cli/command/privileges.py
        5808  01-01-2016 00:00   cli/command/profiles.py
       13973  01-01-2016 00:00   cli/constants.py
         789  01-01-2016 00:00   cli/options/__init__.py
       16669  01-01-2016 00:00   cli/options/option_tree.py
        9355  01-01-2016 00:00   cli/options/parser.py
        8964  01-01-2016 00:00   cli/polaris_cli.py
           0  01-01-2016 00:00   polaris/.keep
         790  01-01-2016 00:00   polaris/__init__.py
           0  01-01-2016 00:00   polaris/catalog/.keep
       11171  01-01-2016 00:00   polaris/catalog/__init__.py
           0  01-01-2016 00:00   polaris/catalog/api/.keep
        1059  01-01-2016 00:00   polaris/catalog/api/__init__.py
      377844  01-01-2016 00:00   polaris/catalog/api/catalog_api.py
       18839  01-01-2016 00:00   polaris/catalog/api/configuration_api.py
       58167  01-01-2016 00:00   polaris/catalog/api/generic_table_api.py
      421177  01-01-2016 00:00   polaris/catalog/api/iceberg_catalog_api.py
       18871  01-01-2016 00:00   
polaris/catalog/api/iceberg_configuration_api.py
       26596  01-01-2016 00:00   polaris/catalog/api/iceberg_o_auth2_api.py
       27127  01-01-2016 00:00   polaris/catalog/api/o_auth2_api.py
      129569  01-01-2016 00:00   polaris/catalog/api/policy_api.py
       28362  01-01-2016 00:00   polaris/catalog/api_client.py
        1442  01-01-2016 00:00   polaris/catalog/api_response.py
       20753  01-01-2016 00:00   polaris/catalog/configuration.py
        7353  01-01-2016 00:00   polaris/catalog/exceptions.py
           0  01-01-2016 00:00   polaris/catalog/models/.keep
       10397  01-01-2016 00:00   polaris/catalog/models/__init__.py
        3557  01-01-2016 00:00   
polaris/catalog/models/add_partition_spec_update.py
        3996  01-01-2016 00:00   polaris/catalog/models/add_schema_update.py
        3525  01-01-2016 00:00   polaris/catalog/models/add_snapshot_update.py
        3570  01-01-2016 00:00   polaris/catalog/models/add_sort_order_update.py
        3588  01-01-2016 00:00   
polaris/catalog/models/add_view_version_update.py
        4070  01-01-2016 00:00   polaris/catalog/models/and_or_expression.py
        4715  01-01-2016 00:00   polaris/catalog/models/applicable_policy.py
        3462  01-01-2016 00:00   polaris/catalog/models/assert_create.py
        3618  01-01-2016 00:00   
polaris/catalog/models/assert_current_schema_id.py
        3643  01-01-2016 00:00   
polaris/catalog/models/assert_default_sort_order_id.py
        3604  01-01-2016 00:00   
polaris/catalog/models/assert_default_spec_id.py
        3651  01-01-2016 00:00   
polaris/catalog/models/assert_last_assigned_field_id.py
        3678  01-01-2016 00:00   
polaris/catalog/models/assert_last_assigned_partition_id.py
        3720  01-01-2016 00:00   
polaris/catalog/models/assert_ref_snapshot_id.py
        3479  01-01-2016 00:00   polaris/catalog/models/assert_table_uuid.py
        3726  01-01-2016 00:00   polaris/catalog/models/assert_view_uuid.py
        3612  01-01-2016 00:00   polaris/catalog/models/assign_uuid_update.py
        3635  01-01-2016 00:00   polaris/catalog/models/async_planning_result.py
        3814  01-01-2016 00:00   polaris/catalog/models/attach_policy_request.py
       12370  01-01-2016 00:00   polaris/catalog/models/base_update.py
        3834  01-01-2016 00:00   polaris/catalog/models/blob_metadata.py
        4129  01-01-2016 00:00   polaris/catalog/models/catalog_config.py
        4586  01-01-2016 00:00   polaris/catalog/models/commit_report.py
        5037  01-01-2016 00:00   polaris/catalog/models/commit_table_request.py
        3852  01-01-2016 00:00   polaris/catalog/models/commit_table_response.py
        4015  01-01-2016 00:00   
polaris/catalog/models/commit_transaction_request.py
        4995  01-01-2016 00:00   polaris/catalog/models/commit_view_request.py
        5077  01-01-2016 00:00   
polaris/catalog/models/completed_planning_result.py
        5264  01-01-2016 00:00   
polaris/catalog/models/completed_planning_with_id_result.py
        6628  01-01-2016 00:00   polaris/catalog/models/content_file.py
        3630  01-01-2016 00:00   polaris/catalog/models/count_map.py
        3430  01-01-2016 00:00   polaris/catalog/models/counter_result.py
        3814  01-01-2016 00:00   
polaris/catalog/models/create_generic_table_request.py
        3710  01-01-2016 00:00   
polaris/catalog/models/create_namespace_request.py
        3714  01-01-2016 00:00   
polaris/catalog/models/create_namespace_response.py
        4182  01-01-2016 00:00   polaris/catalog/models/create_policy_request.py
        5106  01-01-2016 00:00   polaris/catalog/models/create_table_request.py
        4564  01-01-2016 00:00   polaris/catalog/models/create_view_request.py
        5583  01-01-2016 00:00   polaris/catalog/models/data_file.py
        6366  01-01-2016 00:00   polaris/catalog/models/delete_file.py
        3676  01-01-2016 00:00   polaris/catalog/models/detach_policy_request.py
        3730  01-01-2016 00:00   polaris/catalog/models/empty_planning_result.py
        3469  01-01-2016 00:00   
polaris/catalog/models/enable_row_lineage_update.py
        4769  01-01-2016 00:00   polaris/catalog/models/equality_delete_file.py
        3890  01-01-2016 00:00   polaris/catalog/models/error_model.py
       10131  01-01-2016 00:00   polaris/catalog/models/expression.py
        3793  01-01-2016 00:00   
polaris/catalog/models/failed_planning_result.py
        3358  01-01-2016 00:00   polaris/catalog/models/false_expression.py
        7445  01-01-2016 00:00   polaris/catalog/models/fetch_planning_result.py
        3667  01-01-2016 00:00   
polaris/catalog/models/fetch_scan_tasks_request.py
        4933  01-01-2016 00:00   
polaris/catalog/models/fetch_scan_tasks_result.py
        1661  01-01-2016 00:00   polaris/catalog/models/file_format.py
        4642  01-01-2016 00:00   polaris/catalog/models/file_scan_task.py
        4513  01-01-2016 00:00   polaris/catalog/models/generic_table.py
        5610  01-01-2016 00:00   
polaris/catalog/models/get_applicable_policies_response.py
        3884  01-01-2016 00:00   
polaris/catalog/models/get_namespace_response.py
        3673  01-01-2016 00:00   
polaris/catalog/models/iceberg_error_response.py
        3651  01-01-2016 00:00   
polaris/catalog/models/iceberg_error_response1.py
        5474  01-01-2016 00:00   
polaris/catalog/models/list_generic_tables_response.py
        4723  01-01-2016 00:00   
polaris/catalog/models/list_namespaces_response.py
        5458  01-01-2016 00:00   
polaris/catalog/models/list_policies_response.py
        5228  01-01-2016 00:00   polaris/catalog/models/list_tables_response.py
        4294  01-01-2016 00:00   polaris/catalog/models/list_type.py
        3751  01-01-2016 00:00   polaris/catalog/models/literal_expression.py
        4084  01-01-2016 00:00   
polaris/catalog/models/load_credentials_response.py
        3670  01-01-2016 00:00   
polaris/catalog/models/load_generic_table_response.py
        3633  01-01-2016 00:00   polaris/catalog/models/load_policy_response.py
        6636  01-01-2016 00:00   polaris/catalog/models/load_table_result.py
        4505  01-01-2016 00:00   polaris/catalog/models/load_view_result.py
        4603  01-01-2016 00:00   polaris/catalog/models/map_type.py
        3574  01-01-2016 00:00   polaris/catalog/models/metadata_log_inner.py
        5730  01-01-2016 00:00   polaris/catalog/models/metric_result.py
        4576  01-01-2016 00:00   polaris/catalog/models/model_schema.py
        3782  01-01-2016 00:00   polaris/catalog/models/not_expression.py
        3925  01-01-2016 00:00   polaris/catalog/models/notification_request.py
        1686  01-01-2016 00:00   polaris/catalog/models/notification_type.py
        1652  01-01-2016 00:00   polaris/catalog/models/null_order.py
        4179  01-01-2016 00:00   polaris/catalog/models/o_auth_error.py
        4934  01-01-2016 00:00   polaris/catalog/models/o_auth_token_response.py
        3710  01-01-2016 00:00   polaris/catalog/models/partition_field.py
        4059  01-01-2016 00:00   polaris/catalog/models/partition_spec.py
        3764  01-01-2016 00:00   
polaris/catalog/models/partition_statistics_file.py
        1719  01-01-2016 00:00   polaris/catalog/models/plan_status.py
        5680  01-01-2016 00:00   
polaris/catalog/models/plan_table_scan_request.py
        8391  01-01-2016 00:00   
polaris/catalog/models/plan_table_scan_result.py
        5469  01-01-2016 00:00   polaris/catalog/models/policy.py
        4212  01-01-2016 00:00   
polaris/catalog/models/policy_attachment_target.py
        3505  01-01-2016 00:00   polaris/catalog/models/policy_identifier.py
        4982  01-01-2016 00:00   polaris/catalog/models/position_delete_file.py
       17550  01-01-2016 00:00   polaris/catalog/models/primitive_type_value.py
        3537  01-01-2016 00:00   
polaris/catalog/models/register_table_request.py
        3559  01-01-2016 00:00   
polaris/catalog/models/remove_partition_specs_update.py
        3579  01-01-2016 00:00   
polaris/catalog/models/remove_partition_statistics_update.py
        3499  01-01-2016 00:00   
polaris/catalog/models/remove_properties_update.py
        3530  01-01-2016 00:00   
polaris/catalog/models/remove_snapshot_ref_update.py
        3547  01-01-2016 00:00   
polaris/catalog/models/remove_snapshots_update.py
        3543  01-01-2016 00:00   
polaris/catalog/models/remove_statistics_update.py
        4019  01-01-2016 00:00   polaris/catalog/models/rename_table_request.py
        5748  01-01-2016 00:00   
polaris/catalog/models/report_metrics_request.py
        5162  01-01-2016 00:00   polaris/catalog/models/scan_report.py
        5321  01-01-2016 00:00   polaris/catalog/models/scan_tasks.py
        3614  01-01-2016 00:00   
polaris/catalog/models/set_current_schema_update.py
        3665  01-01-2016 00:00   
polaris/catalog/models/set_current_view_version_update.py
        3646  01-01-2016 00:00   
polaris/catalog/models/set_default_sort_order_update.py
        3612  01-01-2016 00:00   
polaris/catalog/models/set_default_spec_update.py
        3745  01-01-2016 00:00   polaris/catalog/models/set_expression.py
        3473  01-01-2016 00:00   polaris/catalog/models/set_location_update.py
        3673  01-01-2016 00:00   
polaris/catalog/models/set_partition_statistics_update.py
        3491  01-01-2016 00:00   polaris/catalog/models/set_properties_update.py
        4525  01-01-2016 00:00   
polaris/catalog/models/set_snapshot_ref_update.py
        3821  01-01-2016 00:00   polaris/catalog/models/set_statistics_update.py
        4601  01-01-2016 00:00   polaris/catalog/models/snapshot.py
        3553  01-01-2016 00:00   polaris/catalog/models/snapshot_log_inner.py
        4325  01-01-2016 00:00   polaris/catalog/models/snapshot_reference.py
        4303  01-01-2016 00:00   polaris/catalog/models/snapshot_summary.py
        1624  01-01-2016 00:00   polaris/catalog/models/sort_direction.py
        3810  01-01-2016 00:00   polaris/catalog/models/sort_field.py
        3995  01-01-2016 00:00   polaris/catalog/models/sort_order.py
        3520  01-01-2016 00:00   
polaris/catalog/models/sql_view_representation.py
        4585  01-01-2016 00:00   polaris/catalog/models/statistics_file.py
        3694  01-01-2016 00:00   polaris/catalog/models/storage_credential.py
        4951  01-01-2016 00:00   polaris/catalog/models/struct_field.py
        4221  01-01-2016 00:00   polaris/catalog/models/struct_type.py
        3526  01-01-2016 00:00   polaris/catalog/models/table_identifier.py
       11336  01-01-2016 00:00   polaris/catalog/models/table_metadata.py
        7060  01-01-2016 00:00   polaris/catalog/models/table_requirement.py
       20637  01-01-2016 00:00   polaris/catalog/models/table_update.py
        4200  01-01-2016 00:00   
polaris/catalog/models/table_update_notification.py
        6053  01-01-2016 00:00   polaris/catalog/models/term.py
        3613  01-01-2016 00:00   polaris/catalog/models/timer_result.py
        2440  01-01-2016 00:00   polaris/catalog/models/token_type.py
        3758  01-01-2016 00:00   polaris/catalog/models/transform_term.py
        3354  01-01-2016 00:00   polaris/catalog/models/true_expression.py
        7384  01-01-2016 00:00   polaris/catalog/models/type.py
        3743  01-01-2016 00:00   polaris/catalog/models/unary_expression.py
        3580  01-01-2016 00:00   
polaris/catalog/models/update_namespace_properties_request.py
        3981  01-01-2016 00:00   
polaris/catalog/models/update_namespace_properties_response.py
        3717  01-01-2016 00:00   polaris/catalog/models/update_policy_request.py
        3565  01-01-2016 00:00   
polaris/catalog/models/upgrade_format_version_update.py
        4136  01-01-2016 00:00   polaris/catalog/models/value_map.py
        3548  01-01-2016 00:00   polaris/catalog/models/view_history_entry.py
        5774  01-01-2016 00:00   polaris/catalog/models/view_metadata.py
        5589  01-01-2016 00:00   polaris/catalog/models/view_representation.py
        5507  01-01-2016 00:00   polaris/catalog/models/view_requirement.py
       11137  01-01-2016 00:00   polaris/catalog/models/view_update.py
        4892  01-01-2016 00:00   polaris/catalog/models/view_version.py
       10295  01-01-2016 00:00   polaris/catalog/rest.py
           0  01-01-2016 00:00   polaris/management/.keep
        6140  01-01-2016 00:00   polaris/management/__init__.py
           0  01-01-2016 00:00   polaris/management/api/.keep
         911  01-01-2016 00:00   polaris/management/api/__init__.py
      377204  01-01-2016 00:00   polaris/management/api/polaris_default_api.py
       28305  01-01-2016 00:00   polaris/management/api_client.py
        1442  01-01-2016 00:00   polaris/management/api_response.py
       19220  01-01-2016 00:00   polaris/management/configuration.py
        7278  01-01-2016 00:00   polaris/management/exceptions.py
           0  01-01-2016 00:00   polaris/management/models/.keep
        5487  01-01-2016 00:00   polaris/management/models/__init__.py
        3594  01-01-2016 00:00   polaris/management/models/add_grant_request.py
        6443  01-01-2016 00:00   
polaris/management/models/authentication_parameters.py
        3576  01-01-2016 00:00   
polaris/management/models/aws_iam_service_identity_info.py
        5645  01-01-2016 00:00   
polaris/management/models/aws_storage_config_info.py
        4105  01-01-2016 00:00   
polaris/management/models/azure_storage_config_info.py
        3630  01-01-2016 00:00   
polaris/management/models/bearer_authentication_parameters.py
        7122  01-01-2016 00:00   polaris/management/models/catalog.py
        3465  01-01-2016 00:00   polaris/management/models/catalog_grant.py
        2982  01-01-2016 00:00   polaris/management/models/catalog_privilege.py
        4000  01-01-2016 00:00   polaris/management/models/catalog_properties.py
        4541  01-01-2016 00:00   polaris/management/models/catalog_role.py
        3788  01-01-2016 00:00   polaris/management/models/catalog_roles.py
        3752  01-01-2016 00:00   polaris/management/models/catalogs.py
        7005  01-01-2016 00:00   
polaris/management/models/connection_config_info.py
        3591  01-01-2016 00:00   
polaris/management/models/create_catalog_request.py
        3723  01-01-2016 00:00   
polaris/management/models/create_catalog_role_request.py
        3975  01-01-2016 00:00   
polaris/management/models/create_principal_request.py
        3759  01-01-2016 00:00   
polaris/management/models/create_principal_role_request.py
        5087  01-01-2016 00:00   polaris/management/models/external_catalog.py
        3461  01-01-2016 00:00   
polaris/management/models/file_storage_config_info.py
        3717  01-01-2016 00:00   
polaris/management/models/gcp_storage_config_info.py
        3719  01-01-2016 00:00   
polaris/management/models/grant_catalog_role_request.py
        3755  01-01-2016 00:00   
polaris/management/models/grant_principal_role_request.py
        5798  01-01-2016 00:00   polaris/management/models/grant_resource.py
        3760  01-01-2016 00:00   polaris/management/models/grant_resources.py
        4598  01-01-2016 00:00   
polaris/management/models/hadoop_connection_config_info.py
        4579  01-01-2016 00:00   
polaris/management/models/hive_connection_config_info.py
        4870  01-01-2016 00:00   
polaris/management/models/iceberg_rest_connection_config_info.py
        3618  01-01-2016 00:00   
polaris/management/models/implicit_authentication_parameters.py
        3581  01-01-2016 00:00   polaris/management/models/namespace_grant.py
        2882  01-01-2016 00:00   
polaris/management/models/namespace_privilege.py
        4005  01-01-2016 00:00   
polaris/management/models/o_auth_client_credentials_parameters.py
        4539  01-01-2016 00:00   polaris/management/models/polaris_catalog.py
        3685  01-01-2016 00:00   polaris/management/models/policy_grant.py
        1825  01-01-2016 00:00   polaris/management/models/policy_privilege.py
        4717  01-01-2016 00:00   polaris/management/models/principal.py
        4845  01-01-2016 00:00   polaris/management/models/principal_role.py
        3748  01-01-2016 00:00   polaris/management/models/principal_roles.py
        4229  01-01-2016 00:00   
polaris/management/models/principal_with_credentials.py
        3610  01-01-2016 00:00   
polaris/management/models/principal_with_credentials_credentials.py
        3785  01-01-2016 00:00   polaris/management/models/principals.py
        3857  01-01-2016 00:00   
polaris/management/models/reset_principal_request.py
        3606  01-01-2016 00:00   
polaris/management/models/revoke_grant_request.py
        5021  01-01-2016 00:00   
polaris/management/models/service_identity_info.py
        4349  01-01-2016 00:00   
polaris/management/models/sig_v4_authentication_parameters.py
        6157  01-01-2016 00:00   
polaris/management/models/storage_config_info.py
        3768  01-01-2016 00:00   polaris/management/models/table_grant.py
        1962  01-01-2016 00:00   polaris/management/models/table_privilege.py
        4441  01-01-2016 00:00   
polaris/management/models/update_catalog_request.py
        3734  01-01-2016 00:00   
polaris/management/models/update_catalog_role_request.py
        3725  01-01-2016 00:00   
polaris/management/models/update_principal_request.py
        3742  01-01-2016 00:00   
polaris/management/models/update_principal_role_request.py
        3756  01-01-2016 00:00   polaris/management/models/view_grant.py
        1771  01-01-2016 00:00   polaris/management/models/view_privilege.py
       10223  01-01-2016 00:00   polaris/management/rest.py
        2510  01-01-2016 00:00   polaris-1.0.0.dist-info/METADATA
         106  01-01-2016 00:00   polaris-1.0.0.dist-info/WHEEL
          48  01-01-2016 00:00   polaris-1.0.0.dist-info/entry_points.txt
       24980  01-01-2016 00:00   polaris-1.0.0.dist-info/RECORD
   ---------                     -------
     2658296                     251 files
     ```



-- 
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: issues-unsubscr...@polaris.apache.org

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

Reply via email to