Skip to content
This repository has been archived by the owner on May 20, 2021. It is now read-only.

Data Preview not working #113

Closed
Paxa opened this issue May 22, 2015 · 7 comments · Fixed by #141
Closed

Data Preview not working #113

Paxa opened this issue May 22, 2015 · 7 comments · Fixed by #141

Comments

@Paxa
Copy link

Paxa commented May 22, 2015

I have some ideas to make it easier to start (I waste my time on it):

  • add prestoCatalog in example
  • add git clone git@github.com:airbnb/airpal.git in readme steps
  • make prestoCoordinator: http://localhost:8080 by default
  • add export NODE_ENV='development'
  • make "browser-sync": "~2.6.5" in src/main/resources/assets/package.json
  • add prebuild package (if possible)
  • add precompiled javascript (if possible)

screen shot 2015-05-22 at 15 05 55

I have more questions:

  1. What is partitions for?
  2. Can we preview sql query result? (I can download, but can not see in browser)
  3. Can we use only one schema from catalog? (tables list is too big)
  4. Why do you choose java? Stack trace is so hard to read
@waltonryan
Copy link

Were you able to figure out how to get the Data Preview to work? I'm using the presto mysql connector.

@Paxa
Copy link
Author

Paxa commented May 28, 2015

Not yet, but I think I have to. I'm using mysql too

@Paxa
Copy link
Author

Paxa commented May 28, 2015

Here is hardcoded hive.

https://github.com/airbnb/airpal/blob/master/src/main/java/com/airbnb/airpal/resources/TablesResource.java#L154

We can see queries in presto (http://localhost:8080), it shown like this:

when I call preview it shows:

SELECT * FROM hive.dbname.users LIMIT 100

As I see from controller - it should pass query attribute connectorId, but it don't send, so server choose hive.

When I call preview it send request to server:

http://localhost:8081/api/table/dbname/users/preview

But should:

http://localhost:8081/api/table/dbname/users/preview?connectorId=mysql

I change it in java, recompile and it works

@Paxa
Copy link
Author

Paxa commented May 28, 2015

This app gives me so much headache, can't wait to rewrite it right for our internal use

@waltonryan
Copy link

Thanks. Could you share the lines of code you added to TablesResource.java to make the change? did you just change Optional.fromNullable(connectorId).or("hive"), to Optional.fromNullable(connectorId).or("mysql"),?

@Paxa
Copy link
Author

Paxa commented May 28, 2015

Yes, that one. In preview method

@bharath0208
Copy link

@Paxa any other better web UI tool to query presto DB apart from Airbnb/airpal.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants