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

support oracle datasource #59

Merged
merged 3 commits into from
Aug 23, 2022
Merged

support oracle datasource #59

merged 3 commits into from
Aug 23, 2022

Conversation

Nicole00
Copy link
Contributor

No description provided.

@randomJoe211 randomJoe211 added the doc affected PR: improvements or additions to documentation label Jan 17, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jan 17, 2022

Codecov Report

Merging #59 (218b859) into master (543659e) will decrease coverage by 0.73%.
The diff coverage is 17.39%.

@@             Coverage Diff              @@
##             master      #59      +/-   ##
============================================
- Coverage     55.05%   54.32%   -0.74%     
  Complexity       76       76              
============================================
  Files            17       17              
  Lines          1344     1364      +20     
  Branches        255      258       +3     
============================================
+ Hits            740      741       +1     
- Misses          474      492      +18     
- Partials        130      131       +1     
Impacted Files Coverage Δ
...la/com/vesoft/exchange/common/config/Configs.scala 65.43% <0.00%> (-1.24%) ⬇️
.../vesoft/exchange/common/config/SourceConfigs.scala 58.99% <30.76%> (-3.80%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@Nicole00 Nicole00 added the do not review PR: not ready for the code review yet label Jan 17, 2022
@Nicole00 Nicole00 removed the do not review PR: not ready for the code review yet label Jan 28, 2022
@Nicole00
Copy link
Contributor Author

close issue #7

@darionyaphet
Copy link
Contributor

Why don't reuse MySQLReader and refine to JDBC Reader ?

@xiajingchun
Copy link
Contributor

xiajingchun commented Jun 2, 2022

oracle configurations need to added to Configs.scala too
oracle.patch.zip

@jievince jievince merged commit 1ec0788 into vesoft-inc:master Aug 23, 2022
@Nicole00 Nicole00 mentioned this pull request Aug 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc affected PR: improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants