parquet-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From tians...@apache.org
Subject git commit: PARQUET-69: Committer doc
Date Wed, 20 Aug 2014 21:01:14 GMT
Repository: incubator-parquet-mr
Updated Branches:
  refs/heads/master 7b415faae -> 45e581028


PARQUET-69: Committer doc

In order to improve turnaround time on reviewing pull requests here are a few guidelines and
informations.

Author: Julien Le Dem <julien@twitter.com>

Closes #23 from julienledem/committer_doc and squashes the following commits:

d4b2e19 [Julien Le Dem] Update REVIEWERS.md
f7d8a5e [Julien Le Dem] Update COMMITTERS.md
b627d50 [Julien Le Dem] Create REVIEWERS.md
0027a98 [Julien Le Dem] Create REVIEWERS.md
6fd49d0 [Julien Le Dem] Create REVIEWERS.md
c6c26e6 [Julien Le Dem] Create REVIEWERS.md
cbbec60 [Julien Le Dem] Create REVIEWERS.md
56b7d3d [Julien Le Dem] Create REVIEWERS.md
a93ddcb [Julien Le Dem] Create REVIEWERS.md
98f1e52 [Julien Le Dem] Create REVIEWERS.md
5d960f4 [Julien Le Dem] Update COMMITTERS.md
7be5f36 [Julien Le Dem] Create REVIEWERS.md
24ef829 [Julien Le Dem] Create REVIEWERS.md
799a067 [Julien Le Dem] Create REVIEWERS.md
8ef1250 [Julien Le Dem] Create REVIEWERS.md
b85f454 [Julien Le Dem] Create REVIEWERS.md
38d1afa [Julien Le Dem] Update COMMITTERS.md
425224d [Julien Le Dem] Update COMMITTERS.md
46249ed [Julien Le Dem] Update COMMITTERS.md
7879de2 [Julien Le Dem] Create COMMITTERS.md


Project: http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/commit/45e58102
Tree: http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/tree/45e58102
Diff: http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/diff/45e58102

Branch: refs/heads/master
Commit: 45e581028ab60949b0303d4a0cfe4eee077bcce8
Parents: 7b415fa
Author: Julien Le Dem <julien@twitter.com>
Authored: Wed Aug 20 14:01:03 2014 -0700
Committer: Tianshuo Deng <tdeng@twitter.com>
Committed: Wed Aug 20 14:01:03 2014 -0700

----------------------------------------------------------------------
 dev/COMMITTERS.md              | 30 ++++++++++++++++++++++++++++++
 parquet-avro/REVIEWERS.md      |  4 ++++
 parquet-cascading/REVIEWERS.md |  4 ++++
 parquet-column/REVIEWERS.md    |  7 +++++++
 parquet-common/REVIEWERS.md    |  7 +++++++
 parquet-encoding/REVIEWERS.md  |  3 +++
 parquet-generator/REVIEWERS.md |  3 +++
 parquet-hadoop/REVIEWERS.md    |  5 +++++
 parquet-hive/REVIEWERS.md      |  5 +++++
 parquet-pig/REVIEWERS.md       |  6 ++++++
 parquet-protobuf/REVIEWERS.md  |  3 +++
 parquet-scrooge/REVIEWERS.md   |  3 +++
 parquet-thrift/REVIEWERS.md    |  6 ++++++
 parquet-tools/REVIEWERS.md     |  3 +++
 14 files changed, 89 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/dev/COMMITTERS.md
----------------------------------------------------------------------
diff --git a/dev/COMMITTERS.md b/dev/COMMITTERS.md
new file mode 100644
index 0000000..032b09c
--- /dev/null
+++ b/dev/COMMITTERS.md
@@ -0,0 +1,30 @@
+# Committers (in aplhabetical order):
+
+| Name               | Apache Id  | github id      | JIRA id     |
+|--------------------|------------|----------------|-------------|
+| Aniket Mokashi     | aniket486  | aniket486      |             |
+| Brock Noland       | brock      | brockn         |             |
+| Chris Aniszczyk    | caniszczyk |                |             |
+| Dmitriy Ryaboy     | dvryaboy   | dvryaboy       |             |
+| Jake Farrell       | jfarrell   |                |             |
+| Jonathan Coveney   | jcoveney   | jcoveney       |             |
+| Julien Le Dem      | julien     | julienledem    | julienledem |
+| Lukas Nalezenec    | lukas      | lukasnalezenec |             |
+| Marcel Kornacker   | marcel     |                |             |
+| Mickael Lacour     | mlacour    | mickaellcr     |             |
+| Nong Li            | nong       | nongli         |             |
+| Remy Pecqueur      | rpecqueur  | Lordshinjo     |             |
+| Ryan Blue          | blue       | rdblue         |             |
+| Tianshuo Deng      | tianshuo   | tsdeng         |             |
+| Tom White          | tomwhite   | tomwhite       |             |
+| Wesley Graham Peck | wesleypeck | wesleypeck     |             |
+
+Reviewing guidelines:
+Committers have the responsibility to give constructive and timely feedback on the pull requests.
+Anybody can give feedback on a pull request but only committers can merge it.
+
+First things to look at in a Pull Request:
+ - Is there a corresponding JIRA, and is it mentioned in the description? If not ask the
contributor to make one.
+ - If a JIRA is open, make sure it is assigned to the contributor. (they need to have the
contributor role here: https://issues.apache.org/jira/plugins/servlet/project-config/PARQUET/roles)
+ - Is it an uncontroversial change that looks good (has apropriate tests and the build is
succesful)? => merge it
+ - Is it something that requires the attention of committers with a specific expertise? =>
mention those committers by their github id in the pull request.

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-avro/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-avro/REVIEWERS.md b/parquet-avro/REVIEWERS.md
new file mode 100644
index 0000000..954e54d
--- /dev/null
+++ b/parquet-avro/REVIEWERS.md
@@ -0,0 +1,4 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Ryan Blue          | blue       | rdblue      |
+| Tom White          | tomwhite   | tomwhite    |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-cascading/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-cascading/REVIEWERS.md b/parquet-cascading/REVIEWERS.md
new file mode 100644
index 0000000..0640f03
--- /dev/null
+++ b/parquet-cascading/REVIEWERS.md
@@ -0,0 +1,4 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Dmitriy Ryaboy     | dvryaboy   | dvryaboy    |
+| Tianshuo Deng      | tianshuo   | tsdeng      |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-column/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-column/REVIEWERS.md b/parquet-column/REVIEWERS.md
new file mode 100644
index 0000000..cad0969
--- /dev/null
+++ b/parquet-column/REVIEWERS.md
@@ -0,0 +1,7 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Aniket Mokashi     | aniket486  | aniket486   |
+| Jonathan Coveney   | jcoveney   | jcoveney    |
+| Julien Le Dem      | julien     | julienledem |
+| Nong Li            | nong       | nongli      |
+| Tianshuo Deng      | tianshuo   | tsdeng      |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-common/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-common/REVIEWERS.md b/parquet-common/REVIEWERS.md
new file mode 100644
index 0000000..cad0969
--- /dev/null
+++ b/parquet-common/REVIEWERS.md
@@ -0,0 +1,7 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Aniket Mokashi     | aniket486  | aniket486   |
+| Jonathan Coveney   | jcoveney   | jcoveney    |
+| Julien Le Dem      | julien     | julienledem |
+| Nong Li            | nong       | nongli      |
+| Tianshuo Deng      | tianshuo   | tsdeng      |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-encoding/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-encoding/REVIEWERS.md b/parquet-encoding/REVIEWERS.md
new file mode 100644
index 0000000..f6fb5c4
--- /dev/null
+++ b/parquet-encoding/REVIEWERS.md
@@ -0,0 +1,3 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Julien Le Dem      | julien     | julienledem |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-generator/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-generator/REVIEWERS.md b/parquet-generator/REVIEWERS.md
new file mode 100644
index 0000000..f6fb5c4
--- /dev/null
+++ b/parquet-generator/REVIEWERS.md
@@ -0,0 +1,3 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Julien Le Dem      | julien     | julienledem |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-hadoop/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-hadoop/REVIEWERS.md b/parquet-hadoop/REVIEWERS.md
new file mode 100644
index 0000000..7b6802c
--- /dev/null
+++ b/parquet-hadoop/REVIEWERS.md
@@ -0,0 +1,5 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Aniket Mokashi     | aniket486  | aniket486   |
+| Julien Le Dem      | julien     | julienledem |
+| Tianshuo Deng      | tianshuo   | tsdeng      |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-hive/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-hive/REVIEWERS.md b/parquet-hive/REVIEWERS.md
new file mode 100644
index 0000000..5b1a89f
--- /dev/null
+++ b/parquet-hive/REVIEWERS.md
@@ -0,0 +1,5 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Brock Noland       | brock      | brockn      |
+| Mickael Lacour     | mlacour    | mickaellcr  |
+| Remy Pecqueur      | rpecqueur  | Lordshinjo  |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-pig/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-pig/REVIEWERS.md b/parquet-pig/REVIEWERS.md
new file mode 100644
index 0000000..495e149
--- /dev/null
+++ b/parquet-pig/REVIEWERS.md
@@ -0,0 +1,6 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Aniket Mokashi     | aniket486  | aniket486   |
+| Dmitriy Ryaboy     | dvryaboy   | dvryaboy    |
+| Jonathan Coveney   | jcoveney   | jcoveney    |
+| Julien Le Dem      | julien     | julienledem |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-protobuf/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-protobuf/REVIEWERS.md b/parquet-protobuf/REVIEWERS.md
new file mode 100644
index 0000000..d787967
--- /dev/null
+++ b/parquet-protobuf/REVIEWERS.md
@@ -0,0 +1,3 @@
+| Name               | Apache Id  | github id      |
+|--------------------|------------|----------------|
+| Lukas Nalezenec    | lukas      | lukasnalezenec |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-scrooge/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-scrooge/REVIEWERS.md b/parquet-scrooge/REVIEWERS.md
new file mode 100644
index 0000000..7b7aa24
--- /dev/null
+++ b/parquet-scrooge/REVIEWERS.md
@@ -0,0 +1,3 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Tianshuo Deng      | tianshuo   | tsdeng      |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-thrift/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-thrift/REVIEWERS.md b/parquet-thrift/REVIEWERS.md
new file mode 100644
index 0000000..2f240b1
--- /dev/null
+++ b/parquet-thrift/REVIEWERS.md
@@ -0,0 +1,6 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Aniket Mokashi     | aniket486  | aniket486   |
+| Dmitriy Ryaboy     | dvryaboy   | dvryaboy    |
+| Julien Le Dem      | julien     | julienledem |
+| Tianshuo Deng      | tianshuo   | tsdeng      |

http://git-wip-us.apache.org/repos/asf/incubator-parquet-mr/blob/45e58102/parquet-tools/REVIEWERS.md
----------------------------------------------------------------------
diff --git a/parquet-tools/REVIEWERS.md b/parquet-tools/REVIEWERS.md
new file mode 100644
index 0000000..8af9721
--- /dev/null
+++ b/parquet-tools/REVIEWERS.md
@@ -0,0 +1,3 @@
+| Name               | Apache Id  | github id   |
+|--------------------|------------|-------------|
+| Wesley Graham Peck | wesleypeck | wesleypeck  |


Mime
View raw message