Movatterモバイル変換

© 2024 NTT DATA Group Corporation 4PostgreSQLコミュニティ (グローバル)The PostgreSQL Global Development GroupPostgreSQL本体を開発しているコミュニティLocal User Groups世界各地 (33ヶ国62地域) にあるPostgreSQLユーザグループPUG = PostgreSQL User GroupEvents世界各地でPostgreSQLに関するカンファレンスなどのイベントが開催Slack, IRC, LinkedIn, etc様々なチャンネルでPostgreSQLユーザが集まってコミュニケーション

© 2024 NTT DATA Group Corporation 5PostgreSQLコミュニティ (日本)日本PostgreSQLユーザ会日本のPostgreSQLユーザグループJPUG = Japan PostgreSQL User Group支部北海道、東北、新潟、名古屋、関西、四国、中国、九州、沖縄文書・書籍関連分科会PostgreSQL関連の文書の翻訳PostgreSQLエンタープライズ・コンソーシアムPostgreSQLの普及推進を目的とする企業コンソーシアムPGECons = PostgreSQL Enterprise Consortium

© 2024 NTT DATA Group Corporation 7PostgreSQLコミュニティ (グローバル)The PostgreSQL Global Development GroupPostgreSQL本体を開発しているコミュニティLocal User Groups世界各地 (35ヶ国67地域) にあるPostgreSQLユーザグループPUG = PostgreSQL User GroupEvents世界各地でPostgreSQLに関するカンファレンスなどのイベントが開催Slack, IRC, LinkedIn, etc様々なチャンネルでPostgreSQLユーザが集まってコミュニケーションPostgreSQL開発コミュニティに参加しよう！

© 2024 NTT DATA Group Corporation 11PostgreSQLコミュニティは何が特異的なのか特定の企業に偏らない開発が進む複数の企業からの参加全体のコンセンサスを取って開発• EDB, Microsoft, AWS, Crunchy Data, NTT, 富士通, SRA OSS, Google Cloud, その他多数• どの企業がメイン、ということもない• コアチームは、独裁的に開発の方向性は決めない• 技術的な方向性は全体で合意のうえ、開発が進められる

© 2024 NTT DATA Group Corporation 12PostgreSQL開発のフロー新機能の提案やバグの報告を行う新機能の必要性やユーザI/F、アーキテクチャ、実現方法、バグ原因、修正方法などを議論する議論結果を踏まえて、新機能やバグ修正をコーディングして、パッチを作成・投稿するパッチをレビューして、不備を改修するコミッタがパッチをPostgreSQL本体に取り込む提案・バグ報告議論パッチ作成レビューコミット繰り返し

© 2024 NTT DATA Group Corporation 14メーリングリストの購読PostgreSQL開発コミュニティでのコミュニケーションはメーリングリストが基本PostgreSQL公式サイトでコミュニティアカウントを作成して、4つのMLを購読pgsql-hackers新機能の提案やバグ修正、開発課題などについて議論するpgsql-bugsユーザからのバグ報告先。報告されたバグやその修正などについて議論するpgsql-docsドキュメントやその問題、改修などについて議論するpgsql-committersコミットされた内容が通知される過去の議論内容は、各メーリングリストのアーカイブから確認可能https://www.postgresql.org/list/

© 2024 NTT DATA Group Corporation 16議論を読むコツ興味あるテーマ・機能・バグの議論から読むOSS-DBの資格取得などを通じてPostgreSQLの体系的な事前知識を得ておく英語の長い議論を手軽に読みたいときは、翻訳サイトなども活用例えば、Shaperで英文を整形して、DeepLで日本語翻訳https://dream-exp.net/shaper/https://www.deepl.com/translator用語・省略語などを理解するhttps://www.crunchydata.com/blog/understanding-the-postgres-hackers-mailing-list

© 2024 NTT DATA Group Corporation 18議論で返信するコツ翻訳サイトなどを活用して、英文を作成例えば、日本語で文章を作成して、DeepLで英語翻訳、Grammarlyで添削https://www.deepl.com/translatorhttps://www.grammarly.com/文を無理に長くしない、箇条書きにする、例を出して説明する他の人のメールやPostgreSQLドキュメント、ソースコメントの英文を参考にする

© 2024 NTT DATA Group Corporation 20PostgreSQL開発のサイクル2024年度 2025年度6 7 8 9 10 11 12 1 2 3 4 5 6 7 8 9 10 11CF CF CF CF CFv18開発v17開発v19開発CF CFBeta / RCBeta / RCFeatureFreeze リリースリリースコミュニティサポートCFCommitFestパッチレビューに集中する期間

© 2024 NTT DATA Group Corporation 22CommitFestのワークフローNeeds reviewレビュー待ちWaiting on Author開発者の対応待ちReady forCommitterコミッタの判断待ちCommittedコミット成功！Moved to next CF次回CFでレビュー継続Returned withFeedback今回CFで十分にレビュー済Rejected提案却下Withdrawn提案撤回パッチ修正レビューパッチ提案レビュー完了差し戻しコミットパッチレビュー中パッチクローズ提案撤回提案却下レビュー十分CF終了パッチ作者レビュアコミッタ

© 2024 NTT DATA Group Corporation 23レビューするパッチを決める興味ある機能や分野のパッチレビューしやすいパッチドキュメントやテストケースのパッチ独立した機能などで影響範囲の小さいパッチ(クライアントツール、contribモジュール、SQL関数など)サイズの小さいパッチ仕様やI/Fなどの内容がコミュニティで合意済のパッチメインのレビュアがいるパッチメインのレビュアによるレビュー内容を勉強しながら、サブのレビュアとして取り組む

© 2024 NTT DATA Group Corporation 26パッチをレビューする提案レビューパッチは適用できるか？ドキュメントやテストを含んでいるか？仕様レビューパッチは本当に必要か？仕様はコミュニティで合意されているか？機能テスト仕様通りに動作するか？クラッシュや想定外のエラーはないか？性能試験パッチにより性能劣化しないか？コーディング・レビューWindowsでも動作するか？コーディング規約に従っているか？アーキテクチャ・レビュー他機能と一貫性のあるアーキテクチャか？https://wiki.postgresql.org/wiki/Reviewing_a_Patch/jaレビューコメントとしてMLで返信する

© 2024 NTT DATA Group Corporation 27パッチを適用、コンパイル、テストする① ソースの入手、コンパイルに必要なライブラリをインストールする$ sudo dnf install git gcc make bison flex readline readline-devel zlib-devel libicu-devel② TAPテスト(*)に必要なライブラリをインストールする$ sudo dnf install perl-CPAN$ sudo cpan -i IPC::Run Test::Simple Time::HiRes Test::Harness③ ドキュメントのコンパイルに必要なライブラリをインストールする$ sudo dnf install docbook-dtds docbook-style-xsl libxslt環境によって必要なライブラリやインストールのコマンドが異なることに注意！上記はRHEL8系環境での実行例(*) PostgreSQLのクライアントツールなどを対象とする追加テスト

© 2024 NTT DATA Group Corporation 28パッチを適用、コンパイル、テストする④ PostgreSQLのgitレポジトリをクローンする$ git clone git://git.postgresql.org/git/postgresql.git⑤ パッチ適用先ブランチをチェックアウトする$ cd postgresql$ git checkout master⑥ パッチ用の新規ブランチを作成する$ git checkout -b xxx$ git branchmaster* xxx基本的に「開発中の最新メジャーバージョン」のmasterブランチがパッチ適用先手元環境だとクローンにかかった時間は約15分レビュー完了後などにパッチ適用を手軽にクリアできるように、パッチ用のブランチを作成するのがおススメ！

© 2024 NTT DATA Group Corporation 29パッチを適用、コンパイル、テストする⑦ パッチを適用する$ patch -p1 -d. < $HOME/xxx.patch⑧ PostgreSQLをコンパイルする$ ./configure --enable-debug --enable-cassert --enable-tap-tests --prefix=$HOME/pgsql/xxx CFLAGS=-O0$ make -j 4$ make installデバッグしやすいようにオプションを指定する--enable-debug--enable-cassert--enable-tap-testsCFLAGS=-O0インストール先のディレクトリを指定する--prefix時間短縮のため複数多重でコンパイルするように-jオプションを指定するのがおススメ！パッチ適用やコンパイルで発生したエラー・警告は、レビューコメントとしてMLで報告する

© 2024 NTT DATA Group Corporation 30パッチを適用、コンパイル、テストする⑨ contribモジュールをコンパイルする$ cd contrib$ make$ make install$ cd ..⑩ ドキュメントをコンパイルする$ make html⑪ リグレッションテストを実行する$ make -j 4 check-worldコンパイルやテストで発生したエラー・警告は、レビューコメントとしてMLで報告するCF登録のパッチは自動的にテストが走り、その結果をPatch Testerから確認できるhttp://cfbot.cputube.org/時間短縮のため複数多重でコンパイルするように-jオプションを指定するのがおススメ！

© 2024 NTT DATA Group Corporation 31パッチを適用、コンパイル、テストする⑫ PostgreSQLを起動する$ cd $HOME/pgsql/xxx$ bin/initdb -D data --locale=C --encoding=UTF8$ bin/pg_ctl -D data start⑬ パッチの新機能をテストする$ bin/psql...⑭ PostgreSQLを停止する$ bin/pg_ctl -D data stop思いつく限りの新機能の使い方を試してみる！例えば、入力のバリエーション (境界値、大規模データ、不正値など)大量実行、複数多重同時実行物理/論理レプリケーション環境での動作起動・停止やテストで発生したエラー・想定外動作は、レビューコメントとしてMLで報告する

© 2024 NTT DATA Group Corporation 32パッチをコーディング・レビューするコーディング規約やエラーメッセージのスタイルガイドに従っているか？https://www.postgresql.jp/document/current/html/source.htmlコメントやドキュメントに内容誤り(古い内容のまま)やTypoはないか？関数名、変数名、マクロ名、ファイル名などの命名が分かりやすく適切か？初期化忘れ、クローズ・解放忘れ、戻り値のチェック忘れなどのバグはないか？セグメンテーションフォルトやアサーションエラーを引き起こすコードパスはないか？無駄、非効率、(必要以上に)複雑すぎる、理解しにくいコードはないか？例えば、以下の観点でコード上気になる点をレビューコメントとしてMLで報告する

© 2024 NTT DATA Group Corporation 38パッチネタを見つける取り組み成果やその関連技術をPostgreSQLに組み込む成果が大きすぎる場合は、細かい技術要素に分けて、少しずつパッチ化できないか考えるコミュニティでの議論やパッチレビューでパッチネタを見つける議論中に新機能のアイデアが出ることや、レビュー中にPostgreSQLの不具合が見つかることが多いPostgreSQLの最新バージョンリリース前後に、そのバージョンの新機能を試すリリース前後は、新機能に不具合が含まれていることが多い

© 2024 NTT DATA Group Corporation 39PostgreSQL開発のフロー新機能の提案やバグの報告を行う新機能の必要性やユーザI/F、アーキテクチャ、実現方法、バグ原因、修正方法などを議論する議論結果を踏まえて、新機能やバグ修正をコーディングして、パッチを作成・投稿するパッチをレビューして、不備を改修するコミッタがパッチをPostgreSQL本体に取り込む提案・バグ報告議論パッチ作成レビューコミット繰り返しパッチネタについてまずはコミュニティで議論して、その結果にもとづいてパッチを作成するのが基本パッチを作成してみないと提案・議論しにくい場合は、パッチを先に作成することも

© 2024 NTT DATA Group Corporation 40パッチを作成する① masterブランチでソースを最新化して、パッチ用のブランチを作成する。$ git checkout master$ git pull$ git checkout -b new-func② ソースを改修して、その改修内容をコミットする。$ emacs ...$ git commit -a -m "Add new func ..."③ masterブランチからパッチ用ブランチへのソース差分をパッチとして作成する。$ git format-patch master$ ls *.patch0001-Add-new-func.patch

© 2024 NTT DATA Group Corporation 41パッチレビューの観点をクリアできているか提案レビューパッチは適用できるか？ドキュメントやテストを含んでいるか？仕様レビューパッチは本当に必要か？仕様はコミュニティで合意されているか？機能テスト仕様通りに動作するか？クラッシュや想定外のエラーはないか？性能試験パッチにより性能劣化しないか？コーディング・レビューWindowsでも動作するか？コーディング規約に従っているか？アーキテクチャ・レビュー他機能と一貫性のあるアーキテクチャか？https://wiki.postgresql.org/wiki/Reviewing_a_Patch/ja

© 2024 NTT DATA Group Corporation 42パッチレビューの観点をクリアできているかコーディング規約やエラーメッセージのスタイルガイドに従っているか？https://www.postgresql.jp/document/current/html/source.htmlコメントやドキュメントに内容誤り(古い内容のまま)やTypoはないか？関数名、変数名、マクロ名、ファイル名などの命名が分かりやすく適切か？初期化忘れ、クローズ・解放忘れ、戻り値のチェック忘れなどのバグはないか？セグメンテーションフォルトやアサーションエラーを引き起こすコードパスはないか？無駄、非効率、(必要以上に)複雑すぎる、理解しにくいコードはないか？

© 2024 NTT DATA Group Corporation 43PostgreSQL開発のサイクル2024年度 2025年度6 7 8 9 10 11 12 1 2 3 4 5 6 7 8 9 10 11CF CF CF CF CFv18開発v17開発v19開発CF CFBeta / RCBeta / RCFeatureFreeze リリースリリースコミュニティサポートCFCommitFestパッチレビューに集中する期間

© 2024 NTT DATA Group Corporation 47参加すると楽しい！PostgreSQLをより深く理解できる・使いこなせるようになる自分の提案に世界中の技術者から反応がある・議論できる・一緒に開発できる・見識も人脈も広がる自分の成果として名前が残り、セルフブランディングになる自分のコードが世界中で使われる・感謝されるコーディングスキルが向上するコミュニティに参加するモチベーション

記載されている会社名、商品名、又はサービス名は、各社の登録商標又は商標です。

Movatterモバイル変換

Change Language

PostgreSQL開発コミュニティに参加しよう！ (オープンデベロッパーズカンファレンス(ODC)2024 発表資料)

Embed presentation

Recommended

More Related Content

What's hot

Similar to PostgreSQL開発コミュニティに参加しよう！ (オープンデベロッパーズカンファレンス(ODC)2024 発表資料)

More from NTT DATA Technology & Innovation

PostgreSQL開発コミュニティに参加しよう！ (オープンデベロッパーズカンファレンス(ODC)2024 発表資料)