You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@dubbo.apache.org by "nanjiek (via GitHub)" <gi...@apache.org> on 2025/11/04 07:59:27 UTC
[D] Enhancements and Standardization Required for Pixiu [dubbo-go-pixiu]
GitHub user nanjiek created a discussion: Enhancements and Standardization Required for Pixiu
### Title: Enhancements and Standardization Required for Pixiu
Pixiu still needs some PRs to improve:
1. **Issue:** The prefix used when logging is not standardized. Some parts of the code use `pixiugatewaycmd`, while others use `dubbo-go-pixiu`.
* **PR 1:** Standardize the logging prefix to `pixiu` across the project.
2. **Issue:** There is no standardized error descriptor. Error descriptors are inconsistent across Pixiu, and there is no standard format for error messages and descriptions.
* **PR 2:** Standardize the error description text format for all errors in Pixiu.
3. **Issue:** The methods for handling replenishment errors vary. Some errors are panicked, while others are just logged.
* **PR 3:** Define and implement error handling guidelines (when to panic vs. when to log).
GitHub link: https://github.com/apache/dubbo-go-pixiu/discussions/793
----
This is an automatically sent email for notifications@dubbo.apache.org.
To unsubscribe, please send an email to: notifications-unsubscribe@dubbo.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org
Re: [D] [GATWEAY] Standardization Pixiu's err report and code [dubbo-go-pixiu]
Posted by "nanjiek (via GitHub)" <gi...@apache.org>.
GitHub user nanjiek edited a discussion: [GATWEAY] Standardization Pixiu's err report and code
### Title: Enhancements and Standardization Required for Pixiu
Pixiu still needs some PRs to improve:
1. **Issue:** The prefix used when logging is not standardized. Some parts of the code use `pixiugatewaycmd`, while others use `dubbo-go-pixiu`.
* **PR 1:** Standardize the logging prefix to `pixiu` across the project.
2. **Issue:** There is no standardized error descriptor. Error descriptors are inconsistent across Pixiu, and there is no standard format for error messages and descriptions.
* **PR 2:** Standardize the error description text format for all errors in Pixiu.
3. **Issue:** The methods for handling replenishment errors vary. Some errors are panicked, while others are just logged.
* **PR 3:** Define and implement error handling guidelines (when to panic vs. when to log).
GitHub link: https://github.com/apache/dubbo-go-pixiu/discussions/793
----
This is an automatically sent email for notifications@dubbo.apache.org.
To unsubscribe, please send an email to: notifications-unsubscribe@dubbo.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org
Re: [D] Standardization Pixiu's err report and code [dubbo-go-pixiu]
Posted by "nanjiek (via GitHub)" <gi...@apache.org>.
GitHub user nanjiek edited a discussion: Standardization Pixiu's err report and code
### Title: Enhancements and Standardization Required for Pixiu
Pixiu still needs some PRs to improve:
1. **Issue:** The prefix used when logging is not standardized. Some parts of the code use `pixiugatewaycmd`, while others use `dubbo-go-pixiu`.
* **PR 1:** Standardize the logging prefix to `pixiu` across the project.
2. **Issue:** There is no standardized error descriptor. Error descriptors are inconsistent across Pixiu, and there is no standard format for error messages and descriptions.
* **PR 2:** Standardize the error description text format for all errors in Pixiu.
3. **Issue:** The methods for handling replenishment errors vary. Some errors are panicked, while others are just logged.
* **PR 3:** Define and implement error handling guidelines (when to panic vs. when to log).
GitHub link: https://github.com/apache/dubbo-go-pixiu/discussions/793
----
This is an automatically sent email for notifications@dubbo.apache.org.
To unsubscribe, please send an email to: notifications-unsubscribe@dubbo.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@dubbo.apache.org
For additional commands, e-mail: notifications-help@dubbo.apache.org