purushah commented on PR #2178: URL: https://github.com/apache/zookeeper/pull/2178#issuecomment-2286375221
Thanks, @tisonkun, for your comment. I did look into existing libraries that offer similar functionality, but I opted against using them for two reasons: 1. ZooKeeper already has `Time.java`, which contains time-based functions, making it a good place to add `parseTimeInterval`. The logic is straightforward, and I don't anticipate it will need changes in the future. 2. I'm always cautious about adding new libraries (especially since none of the existing ones provide a `parseTimeInterval` API) to the classpath, as it could lead to compatibility issues. -- 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: notifications-unsubscr...@zookeeper.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
