You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by GitBox <gi...@apache.org> on 2022/01/15 09:50:02 UTC
[GitHub] [incubator-inlong] gosonzhang opened a new pull request #2163: [INLONG-2160][Improve] Time format conversion using DateTimeFormatter
gosonzhang opened a new pull request #2163:
URL: https://github.com/apache/incubator-inlong/pull/2163
Fixes #2160
This PR revision mainly makes the following changes, and the changes are carried out in the form of equivalent replacement:
1. Construct a DateTimeConvertUtils class, which is responsible for data format conversion operations;
2. Remove the time conversion functions date2yyyyMMddHHmmss and yyyyMMddHHmmss2date in WebParameterUtils, and replace them with the corresponding functions of the DateTimeConvertUtils class;
3. Update other processing logic for time conversion
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-inlong] codecov-commenter commented on pull request #2163: [INLONG-2160][Improve] Time format conversion using DateTimeFormatter
Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on pull request #2163:
URL: https://github.com/apache/incubator-inlong/pull/2163#issuecomment-1013654926
# [Codecov](https://codecov.io/gh/apache/incubator-inlong/pull/2163?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
> Merging [#2163](https://codecov.io/gh/apache/incubator-inlong/pull/2163?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (e083ea2) into [master](https://codecov.io/gh/apache/incubator-inlong/commit/58a3f51f1bac4211ae950f8dddce2d61be3e510c?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (58a3f51) will **increase** coverage by `0.34%`.
> The diff coverage is `81.37%`.
[![Impacted file tree graph](https://codecov.io/gh/apache/incubator-inlong/pull/2163/graphs/tree.svg?width=650&height=150&src=pr&token=1EUK92O9K2&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/incubator-inlong/pull/2163?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
```diff
@@ Coverage Diff @@
## master #2163 +/- ##
============================================
+ Coverage 12.22% 12.57% +0.34%
- Complexity 1154 1194 +40
============================================
Files 413 414 +1
Lines 35243 35282 +39
Branches 5544 5546 +2
============================================
+ Hits 4308 4436 +128
+ Misses 30173 30049 -124
- Partials 762 797 +35
```
| [Impacted Files](https://codecov.io/gh/apache/incubator-inlong/pull/2163?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
|---|---|---|
| [.../apache/inlong/tubemq/corebase/TBaseConstants.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvaW5sb25nL3R1YmVtcS9jb3JlYmFzZS9UQmFzZUNvbnN0YW50cy5qYXZh) | `0.00% <ø> (ø)` | |
| [...pache/inlong/tubemq/corebase/utils/MixedUtils.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvaW5sb25nL3R1YmVtcS9jb3JlYmFzZS91dGlscy9NaXhlZFV0aWxzLmphdmE=) | `0.00% <0.00%> (ø)` | |
| [...che/inlong/tubemq/corebase/utils/TStringUtils.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtY29yZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvaW5sb25nL3R1YmVtcS9jb3JlYmFzZS91dGlscy9UU3RyaW5nVXRpbHMuamF2YQ==) | `5.06% <ø> (+1.26%)` | :arrow_up: |
| [...aster/bdbstore/bdbentitys/BdbBlackGroupEntity.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9pbmxvbmcvdHViZW1xL3NlcnZlci9tYXN0ZXIvYmRic3RvcmUvYmRiZW50aXR5cy9CZGJCbGFja0dyb3VwRW50aXR5LmphdmE=) | `0.00% <0.00%> (ø)` | |
| [...aster/bdbstore/bdbentitys/BdbBrokerConfEntity.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9pbmxvbmcvdHViZW1xL3NlcnZlci9tYXN0ZXIvYmRic3RvcmUvYmRiZW50aXR5cy9CZGJCcm9rZXJDb25mRW50aXR5LmphdmE=) | `56.22% <0.00%> (ø)` | |
| [...store/bdbentitys/BdbConsumeGroupSettingEntity.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9pbmxvbmcvdHViZW1xL3NlcnZlci9tYXN0ZXIvYmRic3RvcmUvYmRiZW50aXR5cy9CZGJDb25zdW1lR3JvdXBTZXR0aW5nRW50aXR5LmphdmE=) | `0.00% <0.00%> (ø)` | |
| [...er/bdbstore/bdbentitys/BdbConsumerGroupEntity.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9pbmxvbmcvdHViZW1xL3NlcnZlci9tYXN0ZXIvYmRic3RvcmUvYmRiZW50aXR5cy9CZGJDb25zdW1lckdyb3VwRW50aXR5LmphdmE=) | `0.00% <0.00%> (ø)` | |
| [...master/bdbstore/bdbentitys/BdbTopicConfEntity.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9pbmxvbmcvdHViZW1xL3NlcnZlci9tYXN0ZXIvYmRic3RvcmUvYmRiZW50aXR5cy9CZGJUb3BpY0NvbmZFbnRpdHkuamF2YQ==) | `51.55% <0.00%> (ø)` | |
| [...he/inlong/tubemq/server/tools/cli/CliProducer.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9pbmxvbmcvdHViZW1xL3NlcnZlci90b29scy9jbGkvQ2xpUHJvZHVjZXIuamF2YQ==) | `0.00% <0.00%> (ø)` | |
| [...r/bdbstore/bdbentitys/BdbClusterSettingEntity.java](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-aW5sb25nLXR1YmVtcS90dWJlbXEtc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9pbmxvbmcvdHViZW1xL3NlcnZlci9tYXN0ZXIvYmRic3RvcmUvYmRiZW50aXR5cy9CZGJDbHVzdGVyU2V0dGluZ0VudGl0eS5qYXZh) | `46.88% <33.33%> (ø)` | |
| ... and [18 more](https://codecov.io/gh/apache/incubator-inlong/pull/2163/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-inlong/pull/2163?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/incubator-inlong/pull/2163?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [58a3f51...e083ea2](https://codecov.io/gh/apache/incubator-inlong/pull/2163?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [incubator-inlong] gosonzhang merged pull request #2163: [INLONG-2160][tubemq] Time format conversion using DateTimeFormatter
Posted by GitBox <gi...@apache.org>.
gosonzhang merged pull request #2163:
URL: https://github.com/apache/incubator-inlong/pull/2163
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@inlong.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org