You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by si...@apache.org on 2018/09/20 17:09:35 UTC
[bookkeeper] branch master updated: [TABLE SERVICE] remove build
warnings "Import xxx.proto but not used"
This is an automated email from the ASF dual-hosted git repository.
sijie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
The following commit(s) were added to refs/heads/master by this push:
new 45321ba [TABLE SERVICE] remove build warnings "Import xxx.proto but not used"
45321ba is described below
commit 45321ba7981fdcdcd3194b14a055487b1a5a227a
Author: Sijie Guo <gu...@gmail.com>
AuthorDate: Thu Sep 20 10:09:30 2018 -0700
[TABLE SERVICE] remove build warnings "Import xxx.proto but not used"
Descriptions of the changes in this PR:
*Motivation*
There are build warnings about `Import xyz.proto but not used`. for example:
```
cluster.proto: warning: Import common.proto but not used.
kv_store.proto: warning: Import kv.proto but not used.
```
*Changes*
Remove unused imports
Author: Sijie Guo <si...@apache.org>
Reviewers: Enrico Olivelli <eo...@gmail.com>
This closes #1685 from sijie/remove_warnings
---
stream/proto/src/main/proto/cluster.proto | 2 --
stream/proto/src/main/proto/kv_store.proto | 1 -
stream/proto/src/main/proto/stream.proto | 2 --
3 files changed, 5 deletions(-)
diff --git a/stream/proto/src/main/proto/cluster.proto b/stream/proto/src/main/proto/cluster.proto
index 1454071..20e9d6d 100644
--- a/stream/proto/src/main/proto/cluster.proto
+++ b/stream/proto/src/main/proto/cluster.proto
@@ -17,8 +17,6 @@
*/
syntax = "proto3";
-import "common.proto";
-
package bookkeeper.proto.cluster;
option java_multiple_files = true;
diff --git a/stream/proto/src/main/proto/kv_store.proto b/stream/proto/src/main/proto/kv_store.proto
index fbaa58f..4625763 100644
--- a/stream/proto/src/main/proto/kv_store.proto
+++ b/stream/proto/src/main/proto/kv_store.proto
@@ -17,7 +17,6 @@
*/
syntax = "proto3";
-import "kv.proto";
import "kv_rpc.proto";
package bookkeeper.proto.kv.store;
diff --git a/stream/proto/src/main/proto/stream.proto b/stream/proto/src/main/proto/stream.proto
index 150e30f..7a59868 100644
--- a/stream/proto/src/main/proto/stream.proto
+++ b/stream/proto/src/main/proto/stream.proto
@@ -17,8 +17,6 @@
*/
syntax = "proto3";
-import "common.proto";
-
package bookkeeper.proto.stream;
option java_multiple_files = true;