You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@systemds.apache.org by GitBox <gi...@apache.org> on 2021/04/15 07:51:49 UTC

[GitHub] [systemds] Baunsgaard commented on pull request #1225: [SYSTEMDS-2916] Build python dist with maven package

Baunsgaard commented on pull request #1225:
URL: https://github.com/apache/systemds/pull/1225#issuecomment-820204693


   Hi @j143 
   
   I both like and dislike the idea. 
   on one hand it makes sense to have a one place build all, but I'm worried about 2 things.
   
   1. Increased package time... (i package the project 10 - 100 times a day) so if it increase this i would not have this change
   2. Confusion about what is inside our builds, since with this we have to manage the python API in our build of the system, rather than only in connection with python.
   
   If you really really like the feature, and intend to use it. I would make it execute under a specific -D like PythonAPI, but then again it does not hook up into anything so there is more work needed if we want to pursue this.
   
   


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

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