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

Adding impyla to support impala as a DB #97

Closed
wants to merge 1 commit into from
Closed

Adding impyla to support impala as a DB #97

wants to merge 1 commit into from

Conversation

Shekharv
Copy link

Adding dependency for impyla (Impala's python based client) so that we can now support impala as a DB

@Shekharv
Adding impyla to support impala as a DB
7e122cd 
Adding dependency for impyla (Impala's python based client) so that we can now support impala as a DB
@mistercrunch
Copy link
Member

Oh, so I don't want to add the specific DB reqs to the global requirements.txt, though the requirements.txt is really just for reference since the actual dependencies for the package are out of setup.py. If I was to add the impala lib, I'd have to add the Oracle / MySQL / Postgres /... ones too potentially and most people have only a few they use in their environments, so it's for each person to install the lib they need depending on the connections they use...

hughhhh referenced this pull request in lyft/incubator-superset Apr 1, 2018
@betodealmeida @lyft-buildnotify-4
Add druiddb to Superset requirements file (#97)
195caf5 
I added `druiddb` to Superset:

1. Added to `requirements.in`.
2. `$ devbox start`
3. `$ control run piptools.compile superset-private`

The submodule `upstream` was also updated, is that expected?
john-bodley added a commit to john-bodley/superset that referenced this pull request Aug 23, 2018
@john-bodley
Merge pull request apache#97 from john-bodley/john-bodley-cherry-pick…
9fd4528 
…-5684

[security] Moving set/merge perm to security manager (apache#5684)
betodealmeida pushed a commit that referenced this pull request Jul 2, 2020
@DiggidyDave
Merge pull request #97 from lyft/mergeApachec0f0d_03112020
f34605e 
reintegrate from c0f0d45 Mar 11, 2020
graceguo-supercat pushed a commit to graceguo-supercat/superset that referenced this pull request Oct 4, 2021
@kristw
update change log and dependency (apache#97)
b5a42de
zhaoyongjie referenced this pull request in zhaoyongjie/incubator-superset Nov 17, 2021
@kristw @zhaoyongjie
update change log and dependency (#97)
ea14e5c
zhaoyongjie referenced this pull request in zhaoyongjie/incubator-superset Nov 24, 2021
@kristw @zhaoyongjie
update change log and dependency (#97)
52414d7
zhaoyongjie referenced this pull request in zhaoyongjie/incubator-superset Nov 25, 2021
@kristw @zhaoyongjie
update change log and dependency (#97)
16e53b8
zhaoyongjie referenced this pull request in zhaoyongjie/incubator-superset Nov 26, 2021
@kristw @zhaoyongjie
update change log and dependency (#97)
b22f9b9
rusackas pushed a commit that referenced this pull request Nov 21, 2023
@sfirke
docs(intro): fix a single broken link (BugHerd #97) (#26039)
f934278
josedev-union pushed a commit to Ortege-xyz/studio that referenced this pull request Jan 22, 2024
@sfirke @josedev-union
docs(intro): fix a single broken link (BugHerd apache#97) (apache#26039)
94edf5d
sfirke added a commit to sfirke/superset that referenced this pull request Mar 22, 2024
@sfirke
docs(intro): fix a single broken link (BugHerd apache#97) (apache#26039)
cf567f4
vinothkumar66 pushed a commit to vinothkumar66/superset that referenced this pull request Nov 11, 2024
@sfirke
docs(intro): fix a single broken link (BugHerd apache#97) (apache#26039)
e2f4636
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.
2 participants
@Shekharv @mistercrunch