WQL array properties: add the trailing separator to match wmi-java
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
Start by reading the WqlRow Javadoc and the array-property rendering, then check how WinRMClient.Builder.arraySeparator(String) configures the separator. Run the project tests and check the array-rendering cases. Done means array elements have the agreed trailing-separator behavior, the empty-array choice is explicit, and WqlRow Javadoc, wql.md and the CHANGELOG describe it.
索引モデルが issue の本文から書いたものです。
説明
Problem
Since 3.0.00 (#189), a WMI array property is rendered by joining its elements with the separator: 192.0.2.10|fe80::1.
wmi-java renders the same array with a trailing separator: WmiStringConverter joins with Collectors.joining(arraySeparator, "", arraySeparator), after stripping the separator from each element, which yields 192.0.2.10|fe80::1| (an empty array yields |).
MetricsHub connectors run the same WQL queries through both libraries (WMI extension on Windows collectors, WinRM extension elsewhere), and their AWK scripts and split computes were written against the wmi-java output. The two libraries must produce the same string.
Proposal
Align winrm-java on wmi-java:
- append the separator after the last element (
a|b|), including for a single element (a|); - strip the separator from each element before joining, as wmi-java does;
- keep
|as the default andWinRMClient.Builder.arraySeparator(String)as the override.
Decide explicitly what an empty array renders as (wmi-java gives |; an empty string may be more useful) and document it in the WqlRow Javadoc, wql.md and the CHANGELOG.
The change only affects array properties, which 2.x truncated to their last element anyway, so no caller can depend on the 3.0.00 form yet.
- 主要言語
- Java
- スター
- 13
- フォーク
- 4
- 平均マージ
- 7時間 38分
- マージ済み PR（30日）
- 14
環境構築
このプロジェクトには開発コンテナ、Dockerfile、コントリビューションガイドがありません。まず README を読み、一般的な手順ははじめてのコントリビューションガイドを参照してください。
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
MetricsHub/winrm-java のほかの issue
-
bug documentation
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
MetricsHub/winrm-java#202 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 78/100
MetricsHub/winrm-java#199 ·
メンテナーはふだん 1 日以内に返信
-
Streaming command fails with a spurious timeout when reconnecting after an idle pause対応中かも @NassimBtk が 1 日前に担当しました。 オープン
MetricsHub/winrm-java#198 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Long-lived `WinRMClient` keeps failing with WSManFault 2150859174 after a terminate Signal fails対応中かも @NassimBtk が 1 日前に担当しました。 オープン
難易度 3/5 1〜2日 初心者へのやさしさ 25/100
MetricsHub/winrm-java#196 · コメント 4 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
enhancement
難易度 5/5 1週間以上 初心者へのやさしさ 30/100
MetricsHub/winrm-java#194 ·
メンテナーはふだん 1 日以内に返信
MetricsHub/winrm-java の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
Netcracker/qubership-integration-platform#1046 ·
メンテナーはふだん 2 日以内に返信
-
`check_java_version()` fails when Java path contains spaces (Windows / Git Bash, `C:\Program Files`)オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
-
Fix Math.ceilDiv wrong result for exact positive divisions対応中かも @pamod-madubashana が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
scala-native/scala-native#5094 ·
メンテナーはふだん 1 日以内に返信
-
[Bug] AI unread message badge counts a batch of new bubbles as one message対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 74/100
apache/rocketmq-dashboard#5784 ·
メンテナーはふだん 3 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
PCL-Community/PCL-CE#3658 ·
メンテナーはふだん 1 日以内に返信