[ 
https://issues.apache.org/jira/browse/GOBBLIN-1611?focusedWorklogId=728079&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-728079
 ]

ASF GitHub Bot logged work on GOBBLIN-1611:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 16/Feb/22 07:43
            Start Date: 16/Feb/22 07:43
    Worklog Time Spent: 10m 
      Work Description: codecov-commenter commented on pull request #3470:
URL: https://github.com/apache/gobblin/pull/3470#issuecomment-1041202094


   # 
[Codecov](https://codecov.io/gh/apache/gobblin/pull/3470?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 
[#3470](https://codecov.io/gh/apache/gobblin/pull/3470?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (dc04999) into 
[master](https://codecov.io/gh/apache/gobblin/commit/655826abeeb3c367a4fe882ab80d1a80c97b7ea1?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 (655826a) will **increase** coverage by `0.00%`.
   > The diff coverage is `n/a`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/gobblin/pull/3470/graphs/tree.svg?width=650&height=150&src=pr&token=4MgURJ0bGc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/gobblin/pull/3470?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    #3470   +/-   ##
   =========================================
     Coverage     46.62%   46.62%           
   + Complexity    10352    10351    -1     
   =========================================
     Files          2076     2076           
     Lines         81032    81032           
     Branches       9044     9044           
   =========================================
   + Hits          37778    37781    +3     
   + Misses        39774    39772    -2     
   + Partials       3480     3479    -1     
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/gobblin/pull/3470?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
 | Coverage Δ | |
   |---|---|---|
   | 
[...gobblin/service/modules/core/GitConfigMonitor.java](https://codecov.io/gh/apache/gobblin/pull/3470/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-Z29iYmxpbi1zZXJ2aWNlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NlcnZpY2UvbW9kdWxlcy9jb3JlL0dpdENvbmZpZ01vbml0b3IuamF2YQ==)
 | `81.35% <0.00%> (-1.70%)` | :arrow_down: |
   | 
[...anagement/copy/replication/ConfigBasedDataset.java](https://codecov.io/gh/apache/gobblin/pull/3470/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-Z29iYmxpbi1kYXRhLW1hbmFnZW1lbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vZGF0YS9tYW5hZ2VtZW50L2NvcHkvcmVwbGljYXRpb24vQ29uZmlnQmFzZWREYXRhc2V0LmphdmE=)
 | `68.87% <0.00%> (ø)` | |
   | 
[.../apache/gobblin/runtime/api/JobExecutionState.java](https://codecov.io/gh/apache/gobblin/pull/3470/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-Z29iYmxpbi1ydW50aW1lL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3J1bnRpbWUvYXBpL0pvYkV4ZWN1dGlvblN0YXRlLmphdmE=)
 | `80.37% <0.00%> (+0.93%)` | :arrow_up: |
   | 
[...lin/util/filesystem/FileSystemInstrumentation.java](https://codecov.io/gh/apache/gobblin/pull/3470/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-Z29iYmxpbi11dGlsaXR5L3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3V0aWwvZmlsZXN5c3RlbS9GaWxlU3lzdGVtSW5zdHJ1bWVudGF0aW9uLmphdmE=)
 | `92.85% <0.00%> (+7.14%)` | :arrow_up: |
   | 
[...he/gobblin/source/PartitionAwareFileRetriever.java](https://codecov.io/gh/apache/gobblin/pull/3470/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-Z29iYmxpbi1jb3JlL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NvdXJjZS9QYXJ0aXRpb25Bd2FyZUZpbGVSZXRyaWV2ZXIuamF2YQ==)
 | `55.55% <0.00%> (+7.40%)` | :arrow_up: |
   
   ------
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/gobblin/pull/3470?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/gobblin/pull/3470?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 
[655826a...dc04999](https://codecov.io/gh/apache/gobblin/pull/3470?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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 728079)
    Time Spent: 20m  (was: 10m)

> Fix a wrong value for writer.codec.type in the document
> -------------------------------------------------------
>
>                 Key: GOBBLIN-1611
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-1611
>             Project: Apache Gobblin
>          Issue Type: Bug
>            Reporter: Kengo Seki
>            Priority: Minor
>          Time Spent: 20m
>  Remaining Estimate: 0h
>
> According to 
> https://gobblin.readthedocs.io/en/latest/user-guide/Configuration-Properties-Glossary/#Writer-Properties,
>  possible values for the writer.codec.type property are NOCOMPRESSION, 
> DEFLATE, SNAPPY.
> But if NOCOMPRESSION is actually specified, the job fails as follows.
> {code}
> 2022-02-16 14:15:33 JST ERROR [ForkExecutor-0] 
> org.apache.gobblin.runtime.fork.Fork [d_t_1644988532694_0] - Fork 0 of task 
> task_EmbeddedGobblin_1644988532694_0 failed.
> org.apache.avro.AvroRuntimeException: Unrecognized codec: nocompression
>       at org.apache.avro.file.CodecFactory.fromString(CodecFactory.java:102)
>       at 
> org.apache.gobblin.util.WriterUtils.getCodecFactory(WriterUtils.java:263)
>       at 
> org.apache.gobblin.writer.AvroHdfsDataWriter.<init>(AvroHdfsDataWriter.java:65)
>       at 
> org.apache.gobblin.writer.AvroDataWriterBuilder.build(AvroDataWriterBuilder.java:45)
>       at 
> org.apache.gobblin.writer.PartitionedDataWriter$4.get(PartitionedDataWriter.java:220)
>       at 
> org.apache.gobblin.writer.PartitionedDataWriter$4.get(PartitionedDataWriter.java:215)
>       at 
> org.apache.gobblin.writer.CloseOnFlushWriterWrapper.<init>(CloseOnFlushWriterWrapper.java:73)
>       at 
> org.apache.gobblin.writer.PartitionedDataWriter.<init>(PartitionedDataWriter.java:214)
>       at org.apache.gobblin.runtime.fork.Fork.buildWriter(Fork.java:571)
>       at 
> org.apache.gobblin.runtime.fork.Fork.buildWriterIfNotPresent(Fork.java:579)
>       at org.apache.gobblin.runtime.fork.Fork.processRecord(Fork.java:525)
>       at 
> org.apache.gobblin.runtime.fork.AsynchronousFork.processRecord(AsynchronousFork.java:103)
>       at 
> org.apache.gobblin.runtime.fork.AsynchronousFork.processRecords(AsynchronousFork.java:86)
>       at org.apache.gobblin.runtime.fork.Fork.run(Fork.java:257)
>       at 
> org.apache.gobblin.util.executors.MDCPropagatingRunnable.run(MDCPropagatingRunnable.java:39)
>       at 
> java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
>       at java.util.concurrent.FutureTask.run(FutureTask.java:266)
>       at 
> java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
>       at 
> java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
>       at java.lang.Thread.run(Thread.java:748)
> {code}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

Reply via email to