Versions Compared

Key

  • This line was added.
  • This line was removed.
  • Formatting was changed.

...

Changes to a CTR module (CTR*) may be merged without waiting for review by a maintainer. However it's advisable to ask a maintainer of a CTR module for review and to wait for some time for his/her feedback.

 


ComponentMaintainers
Ignite Core (data grid, rebalancing, affinity, the rest of internals not covered below)Semen Boikov, Alexey Goncharuk, Yakov Zhdanov, Anton Vinogradov
Marshalling (Binary, Optimized, JDK)Vladimir Ozerov, Denis Magda, Alexey Goncharuk
Discovery & Communication SPIsYakov Zhdanov, Semen Boikov, Denis Magda
Ignite Compute APIYakov Zhdanov, Valentin Kulichenko
Ignite Services APIValentin Kulichenko
Ignite SQL & Text QueriesSergey Vladykin
Ignite Continuous QueriesNikolai Tikhonov, Semen Boikov
Build SystemAnton Vinogradov
Hadoop AcceleratorVladimir Ozerov, Ivan Veselovsky
Spark Shared RDDsAlexey Goncharuk, Andrey Gura
IGFSVladimir Ozerov, Ivan Veselovsky
.Net APIPavel Tupitsyn, Vladimir Ozerov
C++ APIIgor Sapego, Vladimir Ozerov
Streamers (JMS, Flume, Kafka, etc.) CTR*Raúl Kripalani, Roman Shtykh
Docker, Mesos, YARN integrationNikolai Tikhonov
AWS, Google Compute Engine, JClouds integrationDenis Magda, Nikolai Tikhonov
OSGi integrationRaúl Kripalani, Denis Magda
VisorAlexey Kuznetsov
WebSession & WebSession FilterValentin Kulichenko

...

  • If the author of the pull request is not a committer, a committer must be the one to approve the change.
  • If the author of the pull request is a committer, approval from their chosen reviewer is enough. A committer is trusted to choose an appropriate reviewer, even if the reviewer is not a committer.
  • Code modifications can cannot be approved by silence: by lazy consensus (72h) after Dev.List announcementthe patch should be committed if and only if committer is confident with the changes.

Once a pull request is approved, any committer can merge it.

...

1. Create GitHub pull-request

...


 +------------+             +---------------+            +-----------------+

...

Branch should be deleted on branch merged to master or issue cancelled. Committers are in charge of deleting their branches.    


Checklist before push

List of points should be checked before push: 

...