Skip to content

Conversation

@Msksgm Msksgm requested a review from a team as a code owner June 21, 2025 12:17
@opentelemetrybot opentelemetrybot requested review from a team June 21, 2025 12:17
@Msksgm Msksgm force-pushed the feat-content/ja/docs/languages/java-2 branch 3 times, most recently from 7946775 to 413c0a4 Compare June 21, 2025 12:31
Copy link
Contributor

@ymotongpoo ymotongpoo left a comment

Choose a reason for hiding this comment

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

とりあえず目についたところをコメントしました。取りこぼしがあるかもしれません。

<!-- markdownlint-disable blanks-around-fences -->
<?code-excerpt path-base="examples/java/api"?>

APIは、主要な観測可能性シグナル全体にわたってテレメトリーを記録するためのクラスとインターフェースのセットです。
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
APIは、主要な観測可能性シグナル全体にわたってテレメトリーを記録するためのクラスとインターフェースのセットです
APIは、主要なオブザーバビリティシグナル全体にわたってテレメトリーを記録するためのクラスとインターフェースのセットです

Copy link
Member Author

Choose a reason for hiding this comment

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

ありがとうございます。
下記で対応しました。

057f341

[Attributes](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-api/latest/io/opentelemetry/api/common/Attributes.html)は[標準属性定義](/docs/specs/otel/common/#standard-attribute)を表すキー値ペアのバンドルです。
`Attributes`は、OpenTelemetry APIにおける繰り返し出現する概念です。

- [Spans](#span)、スパンイベント、スパンリンクには属性があります
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- [Spans](#span)、スパンイベント、スパンリンクには属性があります
- [スパン](#span)、スパンイベント、スパンリンクには属性があります

ここはカタカナで良いかなと思いました。(前の方にもSpanがあるのでそこも書き換えても良いと思う)

Copy link
Member Author

Choose a reason for hiding this comment

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

ありがとうございます。以下で対応しました。

1bc6075

`Attributes`は、OpenTelemetry APIにおける繰り返し出現する概念です。

- [Spans](#span)、スパンイベント、スパンリンクには属性があります
- [メトリクスインストルメント](#meter)に記録される測定値には属性があります
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- [メトリクスインストルメント](#meter)に記録される測定値には属性があります
- [メトリクス計装](#meter)に記録される測定値には属性があります

Copy link
Contributor

Choose a reason for hiding this comment

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

携帯でレビューしてるので原文を確認して欲しいのですが、リンク先は meter になってるので、「メーター計装」ではないでしょうか

Copy link
Member Author

Choose a reason for hiding this comment

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

原文が metric instruments で、直訳するとメトリクス計装とも翻訳できるため、直訳してしまいました。。。ほかにも探してみます。

https://github.com/open-telemetry/opentelemetry.io/blob/main/content/en/docs/languages/java/api.md?plain=1#L100

Copy link
Member Author

Choose a reason for hiding this comment

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

同様のミスが、もう一点あることを確認し対応しました。

cbf77f4


{{% alert title="Java agent" %}}

Java agentは特別なケースで、`GlobalOpenTelemetry`はagentによって設定されます。
Copy link
Contributor

Choose a reason for hiding this comment

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

エージェント?

Copy link
Member Author

Choose a reason for hiding this comment

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

以下で対応しました。

9494fd8

[SpanBuilder](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-api/latest/io/opentelemetry/api/trace/SpanBuilder.html)と[Span](https://www.javadoc.io/doc/io.opentelemetry/opentelemetry-api/latest/io/opentelemetry/api/trace/Span.html)は、スパンにデータを構築し記録するために使用されます。

`SpanBuilder`は、`Span startSpan()`を呼び出してスパンを開始する前にスパンにデータを追加するために使用されます。
開始後、さまざまな`Span`更新メソッドを呼び出すことでデータを追加/更新できます。開始前に`SpanBuilder`に提供されるデータは、[Samplers](../sdk/#sampler)への入力として提供されます。
Copy link
Contributor

Choose a reason for hiding this comment

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

英語のままなら複数形を取る。あるいはカタカナにする。

Copy link
Member Author

Choose a reason for hiding this comment

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

ありがとうございます。以下で対応しました。

3d15bfd


プログラムカスタマイゼーションは、[サポートされているプロパティ](#environment-variables-and-system-properties)を[プログラマティック設定](#programmatic-configuration)で補完するフックを提供します。

[Spring starter](/docs/zero-code/java/spring-boot-starter/)を使用している場合は、[spring starterプログラマティック設定](/docs/zero-code/java/spring-boot-starter/sdk-configuration/#programmatic-configuration)も参照してください。
Copy link
Contributor

Choose a reason for hiding this comment

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

原文で小文字になっていてもSpringは固有名詞なので大文字始まりに直しておく

Copy link
Member Author

Choose a reason for hiding this comment

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

ありがとうございます。以下の対応で合わせて対応しました。

6a82e97

| `io.opentelemetry.instrumentation.resources.ProcessResourceProvider` | `io.opentelemetry.instrumentation:opentelemetry-resources:{{% param vers.instrumentation %}}-alpha` | プロセスリソース属性を提供します。 |
| `io.opentelemetry.instrumentation.resources.ProcessRuntimeResourceProvider` | `io.opentelemetry.instrumentation:opentelemetry-resources:{{% param vers.instrumentation %}}-alpha` | プロセスランタイムリソース属性を提供します。 |
| `io.opentelemetry.contrib.gcp.resource.GCPResourceProvider` | `io.opentelemetry.contrib:opentelemetry-gcp-resources:{{% param vers.contrib %}}-alpha` | GCPランタイム環境リソース属性を提供します。 |
| `io.opentelemetry.contrib.aws.resource.BeanstalkResourceProvider` | `io.opentelemetry.contrib:opentelemetry-aws-resources:{{% param vers.contrib %}}-alpha` | AWS beanstalkランタイム環境リソース属性を提供します。 |
Copy link
Contributor

Choose a reason for hiding this comment

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

各種サービスに関して、原文が小文字でも、正式名称の表記に揃える

Copy link
Member Author

@Msksgm Msksgm Jun 22, 2025

Choose a reason for hiding this comment

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

すみません、正式名称だと原文に誤り(AWS ec2 -> Amazon EC2)にも見えたため、対応方針に迷いました。
以下のコミットでは、正式名称に合わせる方針で修正しました。不都合あれば、再度修正します。必要に応じて、原文にも PR を出そうと検討しています。

e6119de


@Override
public AggregationTemporality getAggregationTemporality(InstrumentType instrumentType) {
// インストルメント種別の関数として必要な集約時間性を指定
Copy link
Contributor

Choose a reason for hiding this comment

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

計装

Copy link
Contributor

Choose a reason for hiding this comment

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

あるいは実装内での固有名詞としてそのままInstrumentTypeとするか

Copy link
Member Author

@Msksgm Msksgm Jun 22, 2025

Choose a reason for hiding this comment

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

ありがとうございます。後者で対応しました。

a0c56a9


@Override
public Aggregation getDefaultAggregation(InstrumentType instrumentType) {
// オプションでインストルメント種別の関数としてデフォルト集約を指定
Copy link
Contributor

Choose a reason for hiding this comment

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

計装

Copy link
Member Author

@Msksgm Msksgm Jun 22, 2025

Choose a reason for hiding this comment

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

ありがとうございます。以下で対応しました。

5decd35

- [Senders](#senders):異なるHTTP / gRPCクライアントライブラリの抽象化
- OTLPエクスポーターの[認証](#authentication)オプション

#### Senders {#senders}
Copy link
Contributor

Choose a reason for hiding this comment

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

これは固有名詞ではない気がするのでセンダーでいいのでは

Copy link
Member Author

@Msksgm Msksgm Jun 22, 2025

Choose a reason for hiding this comment

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

ありがとうございます。以下で対応しました。

1bd6737

@opentelemetrybot opentelemetrybot requested a review from a team June 21, 2025 23:21
@Msksgm Msksgm force-pushed the feat-content/ja/docs/languages/java-2 branch from efeb6fa to 1bd6737 Compare June 22, 2025 00:16
@Msksgm Msksgm changed the title [ja] docs(i18n): translate some of content/ja/docs/languages/java/*.md files [ja] docs(i18n): translate some of content/en/docs/languages/java/*.md files Jun 24, 2025
@vitorvasc vitorvasc added the sig-approval-missing Co-owning SIG didn't provide an approval label Jul 18, 2025
@Msksgm Msksgm force-pushed the feat-content/ja/docs/languages/java-2 branch from 1bd6737 to db32009 Compare July 18, 2025 12:12
Copy link
Contributor

@ymotongpoo ymotongpoo left a comment

Choose a reason for hiding this comment

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

LGTM!

@ymotongpoo ymotongpoo added docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer and removed sig-approval-missing Co-owning SIG didn't provide an approval labels Jul 24, 2025
@theletterf theletterf added this pull request to the merge queue Jul 25, 2025
Merged via the queue into main with commit 7a84487 Jul 25, 2025
22 checks passed
@theletterf theletterf deleted the feat-content/ja/docs/languages/java-2 branch July 25, 2025 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs-approval-missing Co-owning SIG has provided approval, PR needs approval from docs maintainer lang:ja
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

4 participants