hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sean Busbey (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-16835) Revisit the zookeeper usage at client side
Date Thu, 27 Oct 2016 16:18:58 GMT

    [ https://issues.apache.org/jira/browse/HBASE-16835?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15612387#comment-15612387
] 

Sean Busbey commented on HBASE-16835:
-------------------------------------

{quote}
bq. We don't do protoc as part of our build. It is a step done outside of build to generate
src that is checked in for use by build.
So we need to change the precommit job config? Sean Busbey What's purpose of running protoc
check in precommit?
{quote}

checking that protoc builds still work after a change was added in 2014 by [~tedyu] and [~apurtell]
in HBASE-11375 after we had some changes come in that broke doing a protoc build. The fact
that we don't usually do a protoc run when building makes it more important to check in precommit
(IMHO) because that means we won't discover the problem until far after the change goes in.

Now that we're doing gymnastics around protobuf, we should probably make sure we can do end-to-end
regeneration of all the protobuf stuff in our precommit job, since we have the advantage of
requiring a consistent build environment (via docker).

> Revisit the zookeeper usage at client side
> ------------------------------------------
>
>                 Key: HBASE-16835
>                 URL: https://issues.apache.org/jira/browse/HBASE-16835
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Client, Zookeeper
>    Affects Versions: 2.0.0
>            Reporter: Duo Zhang
>            Assignee: Duo Zhang
>             Fix For: 2.0.0
>
>         Attachments: HBASE-16835.patch
>
>
> Watcher or not.
> Curator or not.
> Keep connection or not.
> ...



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message