ArafatKhan2198 commented on PR #5490: URL: https://github.com/apache/ozone/pull/5490#issuecomment-1824149810
If it's possible we could rephrase the paragraph at the starting of the process() method better clarity and understanding. Please let me know if I have gotten anything wrong in my understanding. ``` // Note on DTOKEN_TABLE handling: // - The DTOKEN_TABLE uniquely utilizes OzoneTokenIdentifier, not a String, as its key. // - Directly casting to String during deserialization would result in errors. // - The omdbLatestUpdateEvents currently maps keys as Strings. Changing this to Object would // significantly affect all ReconOmTasks, hence it remains unchanged for now. // - As DTOKEN_TABLE data is not currently required for synchronization in the Recon OM DB snapshot, // no immediate changes are needed. // - Future integration of DTOKEN_TABLE data will involve storing and retrieving events using Object-type keys. ``` https://github.com/apache/ozone/blob/5e80d8283bd13878d55a2f3a241586930c97f32e/hadoop-ozone/recon/src/main/java/org/apache/hadoop/ozone/recon/tasks/OMDBUpdatesHandler.java#L95 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
