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

Posted to commits@wayang.apache.org by "zkaoudi (via GitHub)" <gi...@apache.org> on 2023/05/01 07:58:21 UTC

[GitHub] [incubator-wayang] zkaoudi commented on pull request #311: Updates for monitoring the runtime metrics of underlying platform (Spark)

zkaoudi commented on PR #311:
URL: https://github.com/apache/incubator-wayang/pull/311#issuecomment-1529455386

   Hi all,
   
   I'm wondering if modifying WayangContext and Job to include the hackIT variables/methods is the "best" way to go. Another alternative would be to create a new class eg. WayangContextWithMonitoring which extends from wayang context and contains whatever is necessary. Same for the Job class. 
   What do you think? @JorgeQuiane @kbeedkar Any thoughts?
   


-- 
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: commits-unsubscribe@wayang.apache.org

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