Skip to content

BOOKKEEPER-31: Project Logo#511

Closed
sijie wants to merge 2 commits into
apache:masterfrom
sijie:BOOKKEEPER-31
Closed

BOOKKEEPER-31: Project Logo#511
sijie wants to merge 2 commits into
apache:masterfrom
sijie:BOOKKEEPER-31

Conversation

@sijie
Copy link
Copy Markdown
Member

@sijie sijie commented Sep 16, 2017

Descriptions of the changes in this PR:

  • upload the bookkeeper logo to site directory (including both svg and png format)
  • add &trademark; to Apache BookKeeper.

Copy link
Copy Markdown
Contributor

@merlimat merlimat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@asfgit
Copy link
Copy Markdown

asfgit commented Sep 16, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/bookkeeper-precommit-pullrequest-docker/21/

Copy link
Copy Markdown
Contributor

@hellostreaming hellostreaming left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1. LGTM. 👍

@jiazhai jiazhai added this to the 4.6.0 milestone Sep 16, 2017
@jiazhai jiazhai closed this in aa03c52 Sep 16, 2017
@sijie sijie deleted the BOOKKEEPER-31 branch July 16, 2018 02:28
athanatos pushed a commit to athanatos/bookkeeper that referenced this pull request Jan 25, 2019
…t match ZOOKEEPER-1910

Existing patch on Jira has been rebased and applied to master.

Replaces apache#511

Author: Andor Molnar <andor@cloudera.com>

Reviewers: phunt@apache.org

Closes apache#522 from anmolnar/ZOOKEEPER-1919 and squashes the following commits:

f81a7608a [Andor Molnar] ZOOKEEPER-1919. Fixed indentation
d6455b062 [Andor Molnar] ZOOKEEPER-1919. Test fixes
769677693 [Andor Molnar] ZOOKEEPER-1919. Changes in C code

Change-Id: I2597854e4b28196a5dc5dc89553ced41537663eb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants