YangSong has posted comments on this change. ( http://gerrit.cloudera.org:8080/14654 )
Change subject: KUDU-2975: Spread WAL across multiple directories ...................................................................... Patch Set 8: (1 comment) Compatibility is an issue right now. How to switch from the "--fs_wal_dir" configuration to the "--fs_wal_dirs"? Compared to the new version, the old one has no "wal_manager_instance" file. That means the WAL directory has no UUID. We may use "kudu fs update" tool to resolve it. Another problem is that tablet's metadata does not have "wal_dir" message. The solution is to go through all the WAL directories, try to find the directory where the tablet located. If found, load the tablet, otherwise mark the tablet failed. Above is my personal idea, I don't know if it will work. http://gerrit.cloudera.org:8080/#/c/14654/8/src/kudu/fs/wal_dirs.h File src/kudu/fs/wal_dirs.h: http://gerrit.cloudera.org:8080/#/c/14654/8/src/kudu/fs/wal_dirs.h@177 PS8, Line 177: // Serializes the DataDirGroupPB associated with the given tablet_id. Some comments in this file are wrong, I will modify it later. -- To view, visit http://gerrit.cloudera.org:8080/14654 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ied496804421d91ff1fa63d49979fde971071506e Gerrit-Change-Number: 14654 Gerrit-PatchSet: 8 Gerrit-Owner: YangSong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: YangSong <[email protected]> Gerrit-Comment-Date: Wed, 13 Nov 2019 07:17:50 +0000 Gerrit-HasComments: Yes
