Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GJ-9] Clean up arrow data source and log4j dependency #10

Merged
merged 2 commits into from
Dec 16, 2021

Conversation

rui-mo
Copy link
Contributor

@rui-mo rui-mo commented Dec 15, 2021

This pr:

  • clean up Arrow data source
  • remove log4j dependency

With this pr, the jar of arrow data source is not needed when submitting Spark job.
Parquet data should be read with "spark.read.parquet" instead of "spark.read.format("arrow").load" used by Arrow data source.
#9

@rui-mo rui-mo changed the title [GJ-9] Clean up arrow data source [GJ-9] Clean up arrow data source and log4j dependency Dec 15, 2021
@rui-mo rui-mo requested a review from weiting-chen December 16, 2021 01:09
@rui-mo
Copy link
Contributor Author

rui-mo commented Dec 16, 2021

@weiting-chen This pr can clean up the log4j dependency.

@rui-mo rui-mo merged commit aabe06c into apache:master Dec 16, 2021
rui-mo added a commit to rui-mo/gazelle-jni that referenced this pull request Dec 16, 2021
* remove arrow datasource

* remove the dependency on log4j
rui-mo added a commit to rui-mo/gazelle-jni that referenced this pull request Dec 20, 2021
* remove arrow datasource

* remove the dependency on log4j
jackyromano pushed a commit to jackyromano/gazelle-jni that referenced this pull request Apr 26, 2022
lviiii pushed a commit to lviiii/gluten that referenced this pull request Jul 25, 2022
* [CI] fix travis

Signed-off-by: Yuan Zhou <[email protected]>

* install arrow data source

Signed-off-by: Yuan Zhou <[email protected]>

* fix unit test

Signed-off-by: Yuan Zhou <[email protected]>
lviiii pushed a commit to lviiii/gluten that referenced this pull request Jul 25, 2022
* [ARROW-DATA-SOURCE-1] Reorganize the source code for the new repository organization

* [ARROW-DATA-SOURCE-1] Make compiling succeed

* [ADS-2] Fix RAM usage CI (apache#3)

* Update tpch.yml

* Update report_ram_log.yml

* Update report_ram_log.yml

* Delete github-ci-fix

* Delete github-ci-fix2

* [Arrow-Data-Source-4]Add mkdocs.yml and update docs

* [ADS-6]  Add utility methods to check leaked Allocators/MemoryPools (apache#7)

Close apache#6

* [NSE-51] Update ArrowWritableColumnVector

* [ADS-9][Parquet] Parquet data source not replaced by default (apache#10)

* [ADS-9][Parquet] Parquet data source not replaced by default

* Code style

* [ADS-13] Validate metric TaskMetrics.peakExecutionMemory for native SQL engine (apache#14)

Closes apache#13

* [ADS-11]Modify title check and automatic link to Issues for PRs (apache#12)

* [ADS-16] Upgrade Arrow version to 3.0.0 (apache#17)

Closes apache#16

* Initialize new repo

* Move arrow data source files to arrow-data-source directory

* move native sql files to native-sql-engine folder

* [NSE-86] Add root pom.xml; Remove native-sql-engine/core/ArrowWritableColumnVector.java (apache#88)

* fix github actions

Signed-off-by: Yuan Zhou <[email protected]>

* fix building & CI

Signed-off-by: Yuan Zhou <[email protected]>

Co-authored-by: Chen Haifeng <[email protected]>
Co-authored-by: zhixingheyi-tian  <[email protected]>
Co-authored-by: Hongze Zhang <[email protected]>
Co-authored-by: HongW2019 <[email protected]>
Co-authored-by: Hongze Zhang <[email protected]>
Co-authored-by: Rui Mo <[email protected]>
jinchengchenghh added a commit to jinchengchenghh/gluten that referenced this pull request Mar 21, 2023
rui-mo pushed a commit to rui-mo/gazelle-jni that referenced this pull request Mar 22, 2023
rui-mo pushed a commit to rui-mo/gazelle-jni that referenced this pull request Mar 23, 2023
yimin-yang added a commit to yimin-yang/gluten that referenced this pull request May 4, 2023
Co-authored-by: yangyimin <[email protected]>
sharkdtu pushed a commit to sharkdtu/gluten that referenced this pull request Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant