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

Add dataset ownership metadata collection #3

Merged
merged 2 commits into from
Dec 10, 2015
Merged

Add dataset ownership metadata collection #3

merged 2 commits into from
Dec 10, 2015

Conversation

czbernard
Copy link
Contributor

Fix bugs and add dataset ownership metadata collection for hdfs

StringBuilder execCmd = new StringBuilder("");
execCmd.append("cd " + remoteDir + ";");
execCmd.append("javac " + JAVA_FILE_NAME + JAVA_EXT + ";");
execCmd.append("java -cp " + CLASSPATH + " " + JAVA_FILE_NAME + " \"" + hiveQuery + "\" " + OUTPUT_FILE_NAME + ";");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As we discussed, it may be better to copy a file to remote, then execute that sql file to avoid escape problem.
In production mode, will read sql from database and store into local file, then do the same steps.

@czbernard
Copy link
Contributor Author

changes that fix escaping issue have been committed.

SunZhaonan added a commit that referenced this pull request Dec 10, 2015
Add dataset ownership metadata collection
@SunZhaonan SunZhaonan merged commit 6a121c5 into datahub-project:linkedin Dec 10, 2015
@rilma rilma mentioned this pull request Nov 21, 2017
hsheth2 referenced this pull request in hsheth2/datahub Feb 16, 2021
* Add codegen

* New architecture + setup file -> console pipeline

* Cleanup source loader

* Basic Kafka metadata source

* Kafka source and extractor

* Add kwargs construct interface

* Fix kafka source unit test

* start working on pipeline test

* kafka datahub sink

* Make myself a profile

* Ingest to datahub from kafka

* Update codegen

* Add restli transport

* Fix bug in restli conversion
hsheth2 referenced this pull request in hsheth2/datahub Feb 16, 2021
* Add codegen

* New architecture + setup file -> console pipeline

* Cleanup source loader

* Basic Kafka metadata source

* Kafka source and extractor

* Add kwargs construct interface

* Fix kafka source unit test

* start working on pipeline test

* kafka datahub sink

* Make myself a profile

* Ingest to datahub from kafka

* Update codegen

* Add restli transport

* Fix bug in restli conversion
shirshanka pushed a commit that referenced this pull request Feb 16, 2021
* Add codegen

* New architecture + setup file -> console pipeline

* Cleanup source loader

* Basic Kafka metadata source

* Kafka source and extractor

* Add kwargs construct interface

* Fix kafka source unit test

* start working on pipeline test

* kafka datahub sink

* Make myself a profile

* Ingest to datahub from kafka

* Update codegen

* Add restli transport

* Fix bug in restli conversion
rslanka pushed a commit that referenced this pull request May 26, 2022
Masterchen09 pushed a commit to Masterchen09/datahub that referenced this pull request Jul 14, 2022
RyanHolstien pushed a commit to RyanHolstien/datahub that referenced this pull request Apr 13, 2023
…luent-schema-registry

fix spring dep issues
aabharti-visa pushed a commit to aabharti-visa/datahub that referenced this pull request Feb 5, 2024
…utes-v1

business-attribute: Created initial version of Business Attribute Scr…
jeff-xu-z pushed a commit to jeff-xu-z/datahub that referenced this pull request Mar 25, 2024
[DPOE-22223] use dev & public artifactory for public images
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.

2 participants