[
https://issues.apache.org/jira/browse/GOBBLIN-1737?focusedWorklogId=822104&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-822104
]
ASF GitHub Bot logged work on GOBBLIN-1737:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 31/Oct/22 20:47
Start Date: 31/Oct/22 20:47
Worklog Time Spent: 10m
Work Description: codecov-commenter commented on PR #3595:
URL: https://github.com/apache/gobblin/pull/3595#issuecomment-1297666762
#
[Codecov](https://codecov.io/gh/apache/gobblin/pull/3595?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
[#3595](https://codecov.io/gh/apache/gobblin/pull/3595?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
(b4d8408) into
[master](https://codecov.io/gh/apache/gobblin/commit/4d991a9bd872a98bc52dda2ec366ad7d77847f9a?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
(4d991a9) will **decrease** coverage by `3.08%`.
> The diff coverage is `n/a`.
```diff
@@ Coverage Diff @@
## master #3595 +/- ##
============================================
- Coverage 46.88% 43.80% -3.09%
+ Complexity 10672 2060 -8612
============================================
Files 2120 408 -1712
Lines 83064 17618 -65446
Branches 9250 2153 -7097
============================================
- Hits 38942 7717 -31225
+ Misses 40554 9042 -31512
+ Partials 3568 859 -2709
```
| [Impacted
Files](https://codecov.io/gh/apache/gobblin/pull/3595?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
| Coverage Δ | |
|---|---|---|
|
[...ervice/modules/core/GobblinServiceGuiceModule.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1zZXJ2aWNlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NlcnZpY2UvbW9kdWxlcy9jb3JlL0dvYmJsaW5TZXJ2aWNlR3VpY2VNb2R1bGUuamF2YQ==)
| | |
|
[...ervice/modules/flow/BaseFlowToJobSpecCompiler.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1zZXJ2aWNlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NlcnZpY2UvbW9kdWxlcy9mbG93L0Jhc2VGbG93VG9Kb2JTcGVjQ29tcGlsZXIuamF2YQ==)
| | |
|
[...blin/service/modules/orchestration/DagManager.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1zZXJ2aWNlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NlcnZpY2UvbW9kdWxlcy9vcmNoZXN0cmF0aW9uL0RhZ01hbmFnZXIuamF2YQ==)
| | |
|
[...e/modules/orchestration/MysqlUserQuotaManager.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1zZXJ2aWNlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NlcnZpY2UvbW9kdWxlcy9vcmNoZXN0cmF0aW9uL015c3FsVXNlclF1b3RhTWFuYWdlci5qYXZh)
| | |
|
[...he/gobblin/converter/avro/AvroToJsonConverter.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1tb2R1bGVzL2dvYmJsaW4tYXZyby1qc29uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NvbnZlcnRlci9hdnJvL0F2cm9Ub0pzb25Db252ZXJ0ZXIuamF2YQ==)
| | |
|
[.../gobblin/source/DatePartitionedJsonFileSource.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1jb3JlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NvdXJjZS9EYXRlUGFydGl0aW9uZWRKc29uRmlsZVNvdXJjZS5qYXZh)
| | |
|
[...apache/gobblin/metrics/event/MultiTimingEvent.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1tZXRyaWNzLWxpYnMvZ29iYmxpbi1tZXRyaWNzLWJhc2Uvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vbWV0cmljcy9ldmVudC9NdWx0aVRpbWluZ0V2ZW50LmphdmE=)
| | |
|
[...in/source/extractor/exception/SchemaException.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1jb3JlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NvdXJjZS9leHRyYWN0b3IvZXhjZXB0aW9uL1NjaGVtYUV4Y2VwdGlvbi5qYXZh)
| | |
|
[...bblin/compaction/mapreduce/orc/OrcValueMapper.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1jb21wYWN0aW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NvbXBhY3Rpb24vbWFwcmVkdWNlL29yYy9PcmNWYWx1ZU1hcHBlci5qYXZh)
| | |
|
[...n/runtime/scheduler/ImmediateJobSpecScheduler.java](https://codecov.io/gh/apache/gobblin/pull/3595/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-Z29iYmxpbi1ydW50aW1lL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3J1bnRpbWUvc2NoZWR1bGVyL0ltbWVkaWF0ZUpvYlNwZWNTY2hlZHVsZXIuamF2YQ==)
| | |
| ... and [1702
more](https://codecov.io/gh/apache/gobblin/pull/3595/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)
| |
:mega: We’re building smart automated test selection to slash your CI/CD
build times. [Learn
more](https://about.codecov.io/iterative-testing/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
Issue Time Tracking
-------------------
Worklog Id: (was: 822104)
Time Spent: 20m (was: 10m)
> Fix bug when using mysql user quota manager
> -------------------------------------------
>
> Key: GOBBLIN-1737
> URL: https://issues.apache.org/jira/browse/GOBBLIN-1737
> Project: Apache Gobblin
> Issue Type: Improvement
> Reporter: Zihan Li
> Priority: Major
> Time Spent: 20m
> Remaining Estimate: 0h
>
> # Change the way we get config to to consistent with other mysql store
> # Only check quota in compiler when the flow compile successful and explain
> flag is not set
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
