Skip to content

Commit

Permalink
ORC-1566: Make Brotli dependency as optional
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?
This PR aims to make the Brotli dependency as optional.

### Why are the changes needed?
Brotli is not required when Brotli codec is not used.

### How was this patch tested?
Pass the CIs.

Closes #1721 from williamhyun/brotli.

Authored-by: William Hyun <[email protected]>
Signed-off-by: Dongjoon Hyun <[email protected]>
  • Loading branch information
williamhyun authored and dongjoon-hyun committed Jan 3, 2024
1 parent bbb1f07 commit b35bb59
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions java/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -207,6 +207,7 @@
<groupId>com.aayushatharva.brotli4j</groupId>
<artifactId>brotli4j</artifactId>
<version>${brotli4j.version}</version>
<optional>true</optional>
</dependency>

<!-- test inter-project -->
Expand Down

0 comments on commit b35bb59

Please sign in to comment.