druid icon indicating copy to clipboard operation
druid copied to clipboard

Update ORC to 1.8.0

Open williamhyun opened this issue 2 years ago • 3 comments

Description

This PR aims to update ORC to 1.8.0. This will bring the latest changes and bug fixes.

https://github.com/apache/orc/releases/tag/v1.8.0 https://orc.apache.org/news/2022/09/03/ORC-1.8.0/

This PR has:

  • [x] been self-reviewed.
  • [x] added or updated version, license, or notice information in licenses.yaml

williamhyun avatar Sep 04 '22 00:09 williamhyun

It seems that CI fails with irrelevant issue during downloading commons-lang:commons-lang:jar:2.4.

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-checkstyle-plugin:3.0.0:check (validate) on project druid: Execution validate of goal org.apache.maven.plugins:maven-checkstyle-plugin:3.0.0:check failed: Plugin org.apache.maven.plugins:maven-checkstyle-plugin:3.0.0 or one of its dependencies could not be resolved: Failed to collect dependencies at org.apache.maven.plugins:maven-checkstyle-plugin:jar:3.0.0 -> org.apache.maven.reporting:maven-reporting-impl:jar:2.3 -> org.apache.maven.doxia:doxia-core:jar:1.2 -> commons-lang:commons-lang:jar:2.4: Failed to read artifact descriptor for commons-lang:commons-lang:jar:2.4: Could not transfer artifact org.apache.commons:commons-parent:pom:9 from/to central (https://repo.maven.apache.org/maven2): transfer failed for https://repo.maven.apache.org/maven2/org/apache/commons/commons-parent/9/commons-parent-9.pom, status: 502 Bad Gateway

dongjoon-hyun avatar Sep 04 '22 02:09 dongjoon-hyun

Retest this please.

dongjoon-hyun avatar Sep 08 '22 18:09 dongjoon-hyun

Could you rebase this PR to the master branch once more, @williamhyun ?

dongjoon-hyun avatar Sep 08 '22 18:09 dongjoon-hyun

@williamhyun looks like license checks is failing with the following missing license declaration.

druid_module: orc-extensions, groupId: org.threeten, artifactId: threeten-extra, version: 1.7.1, license: BSD-3-Clause License

xvrl avatar Dec 15 '23 17:12 xvrl

This pull request has been marked as stale due to 60 days of inactivity. It will be closed in 4 weeks if no further activity occurs. If you think that's incorrect or this pull request should instead be reviewed, please simply write any comment. Even if closed, you can still revive the PR at any time or discuss it on the [email protected] list. Thank you for your contributions.

github-actions[bot] avatar Feb 14 '24 00:02 github-actions[bot]

This pull request/issue has been closed due to lack of activity. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

github-actions[bot] avatar Mar 13 '24 00:03 github-actions[bot]