Andrew Wong has uploaded this change for review. ( http://gerrit.cloudera.org:8080/14856
Change subject: fs: make some fields optional ...................................................................... fs: make some fields optional Some fields in fs.proto were defined as required. This isn't good practice; I expect these definitions to be reused for the implementation of a new directory manager for WALs, so let's make them optional. I've only updated the ones I expect to be relevant for adding a new directory manager (i.e. those that involve path instances, etc.). Is this backwards compatible? Assuming the DataDirManager continues to use the changed fields (which I expect), yes. Change-Id: I5378391a94e5faa54a11a277d9191d151e0225d5 --- M src/kudu/fs/fs.proto 1 file changed, 6 insertions(+), 6 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/14856/1 -- To view, visit http://gerrit.cloudera.org:8080/14856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I5378391a94e5faa54a11a277d9191d151e0225d5 Gerrit-Change-Number: 14856 Gerrit-PatchSet: 1 Gerrit-Owner: Andrew Wong <[email protected]>
