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.

Sign up for GitHub

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

Jump to bottom

Make HadoopTableOperations public with protected constructor. #10

Closed
wants to merge 1 commit into from
Closed

Make HadoopTableOperations public with protected constructor. #10

wants to merge 1 commit into from

Conversation

mccheah
Copy link
Contributor

@mccheah mccheah commented Nov 26, 2018

Allows custom implementations that share most of the behavior, but protected constructor prevents it from being instantiated improperly.

@mccheah
Make HadoopTableOperations public with protected constructor.
f9f6df3 
Allows custom implementations that share most of the behavior, but protected constructor prevents it from being instantiated improperly.
@mccheah
Copy link
Contributor Author

mccheah commented Nov 27, 2018

Actually will revisit bundling this with #12 and #13, it doesn't make much sense in isolation.

@mccheah mccheah closed this Nov 27, 2018
rdsr pushed a commit to rdsr/incubator-iceberg that referenced this pull request Oct 26, 2019
@shardulm94 @rdsr
Add custom hive catalog to not override existing Hive metadata (apach…
7ef5465 
…e#10)

Add custom hive catalog to not override existing Hive metadata
rdsr pushed a commit to rdsr/incubator-iceberg that referenced this pull request Mar 13, 2020
@shardulm94
Add custom hive catalog to not override existing Hive metadata (apach…
3c7081b 
…e#10)

Add custom hive catalog to not override existing Hive metadata

Fail early with a proper exception if the metadata file is not existing

Simplify CustomHiveCatalog (apache#22)
rdblue referenced this pull request in rdblue/iceberg May 26, 2020
@rdblue
Updates (#10)
73fab7a 
Merge master, tidy up, fix shadow jar publishing
HotSushi pushed a commit to HotSushi/iceberg that referenced this pull request Jul 31, 2020
@shardulm94 @HotSushi
Pull new changes from upstream Apache Iceberg
2d06d27 
Hive Catalog: Add a hive catalog that does not override existing Hive metadata (apache#10)

Add custom hive catalog to not override existing Hive metadata

Fail early with a proper exception if the metadata file is not existing

Simplify CustomHiveCatalog (apache#22)
bkahloon pushed a commit to bkahloon/iceberg that referenced this pull request Feb 27, 2021
@shardulm94
Hive Catalog: Add a hive catalog that does not override existing Hive…
6578106 
… metadata (apache#10)

Add custom hive catalog to not override existing Hive metadata

Fail early with a proper exception if the metadata file is not existing

Simplify CustomHiveCatalog (apache#22)
SinghAsDev referenced this pull request in SinghAsDev/iceberg Jun 23, 2022
@SinghAsDev
Fix concurrent transactions overwriting commits by adding hive lock h…
bb47d3d 
…eartbeats. (#10)

* Add heartbeats for hive locks

* Remove redundant import

* Add a test case

* Fix test

* remove variance from test

* Minor cleanup
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@mccheah