nsivabalan commented on PR #8759: URL: https://github.com/apache/hudi/pull/8759#issuecomment-1558187571
@SteNicholas : I get your intention. we did take a look at the active timeline methods where we do the transition. As of now, HoodieActive timeline is lightweight and does not have much dependency w/ other hudi entities as much as possible. we don't want to add dep to HoodieMetrics within Active timeline. Especially given we wanted to add this only for compaction. If demand arises that we want to emit this metric for all operations, we can think about it. but for now, may be we can keep it one of and leave it as is. let us know what do you think. -- 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]
