Will Berkeley has posted comments on this change. Change subject: Link to troubleshooting docs for two common problems ......................................................................
Patch Set 3: (7 comments) http://gerrit.cloudera.org:8080/#/c/8093/2/docs/troubleshooting.adoc File docs/troubleshooting.adoc: PS2, Line 299: producing backpressure, > Don't think this one made it into PS3? Done http://gerrit.cloudera.org:8080/#/c/8093/3/docs/troubleshooting.adoc File docs/troubleshooting.adoc: PS3, Line 329: If in the latter situation, consult the : link:administration.html#disk_failure_recovery[Recovering from Disk Failure] documentation. > Just updated this so linking to administration.adoc#change_dir_config proba Nice. Thanks! http://gerrit.cloudera.org:8080/#/c/8093/3/src/kudu/fs/fs_manager.cc File src/kudu/fs/fs_manager.cc: Line 56: #include "kudu/util/website_util.h" > Need to resort. Done Line 56: #include "kudu/util/website_util.h" > warning: #includes are not sorted properly [llvm-include-order] Done http://gerrit.cloudera.org:8080/#/c/8093/3/src/kudu/tserver/tablet_service.cc File src/kudu/tserver/tablet_service.cc: Line 90: #include "kudu/util/website_util.h" > warning: #includes are not sorted properly [llvm-include-order] Done Line 90: #include "kudu/util/website_util.h" > And here. Done http://gerrit.cloudera.org:8080/#/c/8093/3/src/kudu/util/CMakeLists.txt File src/kudu/util/CMakeLists.txt: Line 152: website_util.cc > And here. Done -- To view, visit http://gerrit.cloudera.org:8080/8093 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ida7e1495e4ba68f9e9b4d424650d84c0019c9b0f Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-HasComments: Yes
