Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/14856
to look at the new patch set (#2).
Change subject: fs: make some fields optional
......................................................................
fs: make some fields optional
Some fields involved in directory tracking were defined as required.
This isn't good practice, and 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).
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, 4 insertions(+), 4 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/14856/2
--
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: newpatchset
Gerrit-Change-Id: I5378391a94e5faa54a11a277d9191d151e0225d5
Gerrit-Change-Number: 14856
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
