[jira] [Commented] (GRIFFIN-202) Allow viewing of raw measure details from UI
[ https://issues.apache.org/jira/browse/GRIFFIN-202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641359#comment-16641359 ] Lionel Liu commented on GRIFFIN-202: This could be useful for users, I agree on it. The textual format of measure details just describes the measure, which is something like a template of its jobs. The textual format might not provide any more information than current table format, no matter in JSON or YAML, but in this way, we can consider to enable users create customized measurements in textual format. That would be a great enhancement. Considering this, in the textual format, I prefer only showing the necessary fields of measure, just the same as the measure creation. > Allow viewing of raw measure details from UI > > > Key: GRIFFIN-202 > URL: https://issues.apache.org/jira/browse/GRIFFIN-202 > Project: Griffin (Incubating) > Issue Type: Improvement >Reporter: Nikolay Sokolov >Priority: Minor > > Sometimes it's desirable to see raw JSON of specific measure. This is > especially useful when custom measurements are created from API. > Two implementations are possible (one or even both can be implemented): > * show link with URL of measure details API on UI > * add section under "Mapping Rules", showing formatted plain text of API > response on UI -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Resolved] (GRIFFIN-195) [UI] Don't list all table objects from UI
[ https://issues.apache.org/jira/browse/GRIFFIN-195?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Lionel Liu resolved GRIFFIN-195. Resolution: Fixed Fix Version/s: 1.0.0-incubating Issue resolved by pull request 431 [https://github.com/apache/incubator-griffin/pull/431] > [UI] Don't list all table objects from UI > - > > Key: GRIFFIN-195 > URL: https://issues.apache.org/jira/browse/GRIFFIN-195 > Project: Griffin (Incubating) > Issue Type: Sub-task >Reporter: Nikolay Sokolov >Priority: Major > Fix For: 1.0.0-incubating > > > Listing all table objects when rendering list of tables to profile takes lots > of time, and adds significant latency even if response is cached, when number > of tables is big and/or schema contains lots of columns. > Two solutions are possible: > 1) Without GRIFFIN-194, use DB list API and then list tables API is called > for each DB, in order to collect all tables. When table is clicked on, > request to get table details can be made. > 2) With GRIFFIN-194, DB list followed by number of table list operations can > be replaced by "list all table names in all databases" operation. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (GRIFFIN-195) [UI] Don't list all table objects from UI
[ https://issues.apache.org/jira/browse/GRIFFIN-195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641353#comment-16641353 ] ASF GitHub Bot commented on GRIFFIN-195: Github user asfgit closed the pull request at: https://github.com/apache/incubator-griffin/pull/431 > [UI] Don't list all table objects from UI > - > > Key: GRIFFIN-195 > URL: https://issues.apache.org/jira/browse/GRIFFIN-195 > Project: Griffin (Incubating) > Issue Type: Sub-task >Reporter: Nikolay Sokolov >Priority: Major > > Listing all table objects when rendering list of tables to profile takes lots > of time, and adds significant latency even if response is cached, when number > of tables is big and/or schema contains lots of columns. > Two solutions are possible: > 1) Without GRIFFIN-194, use DB list API and then list tables API is called > for each DB, in order to collect all tables. When table is clicked on, > request to get table details can be made. > 2) With GRIFFIN-194, DB list followed by number of table list operations can > be replaced by "list all table names in all databases" operation. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] incubator-griffin pull request #431: [GRIFFIN-195] Don't list all tables fro...
Github user asfgit closed the pull request at: https://github.com/apache/incubator-griffin/pull/431 ---
[jira] [Commented] (GRIFFIN-195) [UI] Don't list all table objects from UI
[ https://issues.apache.org/jira/browse/GRIFFIN-195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641352#comment-16641352 ] ASF GitHub Bot commented on GRIFFIN-195: Github user bhlx3lyx7 commented on the issue: https://github.com/apache/incubator-griffin/pull/431 Cool, seems good to me, I can merge it now. > [UI] Don't list all table objects from UI > - > > Key: GRIFFIN-195 > URL: https://issues.apache.org/jira/browse/GRIFFIN-195 > Project: Griffin (Incubating) > Issue Type: Sub-task >Reporter: Nikolay Sokolov >Priority: Major > > Listing all table objects when rendering list of tables to profile takes lots > of time, and adds significant latency even if response is cached, when number > of tables is big and/or schema contains lots of columns. > Two solutions are possible: > 1) Without GRIFFIN-194, use DB list API and then list tables API is called > for each DB, in order to collect all tables. When table is clicked on, > request to get table details can be made. > 2) With GRIFFIN-194, DB list followed by number of table list operations can > be replaced by "list all table names in all databases" operation. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] incubator-griffin issue #431: [GRIFFIN-195] Don't list all tables from UI
Github user bhlx3lyx7 commented on the issue: https://github.com/apache/incubator-griffin/pull/431 Cool, seems good to me, I can merge it now. ---
[jira] [Commented] (GRIFFIN-195) [UI] Don't list all table objects from UI
[ https://issues.apache.org/jira/browse/GRIFFIN-195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641329#comment-16641329 ] ASF GitHub Bot commented on GRIFFIN-195: Github user chemikadze commented on the issue: https://github.com/apache/incubator-griffin/pull/431 @bhlx3lyx7 pushed the same for accuracy > [UI] Don't list all table objects from UI > - > > Key: GRIFFIN-195 > URL: https://issues.apache.org/jira/browse/GRIFFIN-195 > Project: Griffin (Incubating) > Issue Type: Sub-task >Reporter: Nikolay Sokolov >Priority: Major > > Listing all table objects when rendering list of tables to profile takes lots > of time, and adds significant latency even if response is cached, when number > of tables is big and/or schema contains lots of columns. > Two solutions are possible: > 1) Without GRIFFIN-194, use DB list API and then list tables API is called > for each DB, in order to collect all tables. When table is clicked on, > request to get table details can be made. > 2) With GRIFFIN-194, DB list followed by number of table list operations can > be replaced by "list all table names in all databases" operation. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] incubator-griffin issue #431: [GRIFFIN-195] Don't list all tables from UI
Github user chemikadze commented on the issue: https://github.com/apache/incubator-griffin/pull/431 @bhlx3lyx7 pushed the same for accuracy ---
[jira] [Commented] (GRIFFIN-202) Allow viewing of raw measure details from UI
[ https://issues.apache.org/jira/browse/GRIFFIN-202?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641323#comment-16641323 ] Nikolay Sokolov commented on GRIFFIN-202: - Probably showing rules as YAML rather than JSON might be even better. That's not a true "plain text of API response", however should make it more readable. Probably we could have json/yaml switch on UI. > Allow viewing of raw measure details from UI > > > Key: GRIFFIN-202 > URL: https://issues.apache.org/jira/browse/GRIFFIN-202 > Project: Griffin (Incubating) > Issue Type: Improvement >Reporter: Nikolay Sokolov >Priority: Minor > > Sometimes it's desirable to see raw JSON of specific measure. This is > especially useful when custom measurements are created from API. > Two implementations are possible (one or even both can be implemented): > * show link with URL of measure details API on UI > * add section under "Mapping Rules", showing formatted plain text of API > response on UI -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Closed] (GRIFFIN-199) [UI] Non-descriptive message when job creation failed
[
https://issues.apache.org/jira/browse/GRIFFIN-199?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Nikolay Sokolov closed GRIFFIN-199.
---
Resolution: Fixed
> [UI] Non-descriptive message when job creation failed
> -
>
> Key: GRIFFIN-199
> URL: https://issues.apache.org/jira/browse/GRIFFIN-199
> Project: Griffin (Incubating)
> Issue Type: Bug
>Reporter: Nikolay Sokolov
>Priority: Minor
>
> When job creation fails, non-descriptive pop-up is shown with message "Error:
> Error when creating job!".
> At the same time, API error contains "code", "error" and "message" fields,
> which could be useful part of this message:
> {code:none}
> {
> "timestamp":1537912602882,
> "status":400,
> "error":"Bad Request",
> "code":"40010",
> "message":"Property 'cron.expression' is invalid",
> "path":"/api/v1/jobs"
> }
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
[jira] [Commented] (GRIFFIN-199) [UI] Non-descriptive message when job creation failed
[
https://issues.apache.org/jira/browse/GRIFFIN-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641310#comment-16641310
]
ASF GitHub Bot commented on GRIFFIN-199:
Github user chemikadze commented on the issue:
https://github.com/apache/incubator-griffin/pull/423
Closes GRIFFIN-199
> [UI] Non-descriptive message when job creation failed
> -
>
> Key: GRIFFIN-199
> URL: https://issues.apache.org/jira/browse/GRIFFIN-199
> Project: Griffin (Incubating)
> Issue Type: Bug
>Reporter: Nikolay Sokolov
>Priority: Minor
>
> When job creation fails, non-descriptive pop-up is shown with message "Error:
> Error when creating job!".
> At the same time, API error contains "code", "error" and "message" fields,
> which could be useful part of this message:
> {code:none}
> {
> "timestamp":1537912602882,
> "status":400,
> "error":"Bad Request",
> "code":"40010",
> "message":"Property 'cron.expression' is invalid",
> "path":"/api/v1/jobs"
> }
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
[jira] [Commented] (GRIFFIN-199) [UI] Non-descriptive message when job creation failed
[
https://issues.apache.org/jira/browse/GRIFFIN-199?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641311#comment-16641311
]
Nikolay Sokolov commented on GRIFFIN-199:
-
Closed by https://github.com/apache/incubator-griffin/pull/423
> [UI] Non-descriptive message when job creation failed
> -
>
> Key: GRIFFIN-199
> URL: https://issues.apache.org/jira/browse/GRIFFIN-199
> Project: Griffin (Incubating)
> Issue Type: Bug
>Reporter: Nikolay Sokolov
>Priority: Minor
>
> When job creation fails, non-descriptive pop-up is shown with message "Error:
> Error when creating job!".
> At the same time, API error contains "code", "error" and "message" fields,
> which could be useful part of this message:
> {code:none}
> {
> "timestamp":1537912602882,
> "status":400,
> "error":"Bad Request",
> "code":"40010",
> "message":"Property 'cron.expression' is invalid",
> "path":"/api/v1/jobs"
> }
> {code}
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
[GitHub] incubator-griffin issue #423: Adjust error msg when failing to create job
Github user chemikadze commented on the issue: https://github.com/apache/incubator-griffin/pull/423 Closes GRIFFIN-199 ---
[jira] [Created] (GRIFFIN-203) "Plaintext mode" for measure creation
Nikolay Sokolov created GRIFFIN-203: --- Summary: "Plaintext mode" for measure creation Key: GRIFFIN-203 URL: https://issues.apache.org/jira/browse/GRIFFIN-203 Project: Griffin (Incubating) Issue Type: New Feature Reporter: Nikolay Sokolov Creating custom rules from API might be cumbersome -- body should be prepared outside of UI, and then submitted using HTTP call. To make user's life easier, it would be useful to allow measure creation by editing JSON directly on UI. Viewing side of this feature would be GRIFFIN-202. Also, experience-wise, JSON might not be the best option for complex spark-sql rules. Possible solution to that would be allowing to write YAML representation instead of JSON, and then either submitting YAML body or converting from YAML to JSON on UI side before submission. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Commented] (GRIFFIN-195) [UI] Don't list all table objects from UI
[ https://issues.apache.org/jira/browse/GRIFFIN-195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641309#comment-16641309 ] ASF GitHub Bot commented on GRIFFIN-195: Github user chemikadze commented on the issue: https://github.com/apache/incubator-griffin/pull/431 @bhlx3lyx7 my thoughts exactly, will push in a few minutes > [UI] Don't list all table objects from UI > - > > Key: GRIFFIN-195 > URL: https://issues.apache.org/jira/browse/GRIFFIN-195 > Project: Griffin (Incubating) > Issue Type: Sub-task >Reporter: Nikolay Sokolov >Priority: Major > > Listing all table objects when rendering list of tables to profile takes lots > of time, and adds significant latency even if response is cached, when number > of tables is big and/or schema contains lots of columns. > Two solutions are possible: > 1) Without GRIFFIN-194, use DB list API and then list tables API is called > for each DB, in order to collect all tables. When table is clicked on, > request to get table details can be made. > 2) With GRIFFIN-194, DB list followed by number of table list operations can > be replaced by "list all table names in all databases" operation. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[jira] [Created] (GRIFFIN-202) Allow viewing of raw measure details from UI
Nikolay Sokolov created GRIFFIN-202: --- Summary: Allow viewing of raw measure details from UI Key: GRIFFIN-202 URL: https://issues.apache.org/jira/browse/GRIFFIN-202 Project: Griffin (Incubating) Issue Type: Improvement Reporter: Nikolay Sokolov Sometimes it's desirable to see raw JSON of specific measure. This is especially useful when custom measurements are created from API. Two implementations are possible (one or even both can be implemented): * show link with URL of measure details API on UI * add section under "Mapping Rules", showing formatted plain text of API response on UI -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] incubator-griffin issue #431: [GRIFFIN-195] Don't list all tables from UI
Github user bhlx3lyx7 commented on the issue: https://github.com/apache/incubator-griffin/pull/431 I've tested this PR, the filtering feature works good, but only in profiling creation step. Would you add it in accuracy creation steps as well? ---
[jira] [Commented] (GRIFFIN-195) [UI] Don't list all table objects from UI
[ https://issues.apache.org/jira/browse/GRIFFIN-195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641299#comment-16641299 ] ASF GitHub Bot commented on GRIFFIN-195: Github user bhlx3lyx7 commented on the issue: https://github.com/apache/incubator-griffin/pull/431 I've tested this PR, the filtering feature works good, but only in profiling creation step. Would you add it in accuracy creation steps as well? > [UI] Don't list all table objects from UI > - > > Key: GRIFFIN-195 > URL: https://issues.apache.org/jira/browse/GRIFFIN-195 > Project: Griffin (Incubating) > Issue Type: Sub-task >Reporter: Nikolay Sokolov >Priority: Major > > Listing all table objects when rendering list of tables to profile takes lots > of time, and adds significant latency even if response is cached, when number > of tables is big and/or schema contains lots of columns. > Two solutions are possible: > 1) Without GRIFFIN-194, use DB list API and then list tables API is called > for each DB, in order to collect all tables. When table is clicked on, > request to get table details can be made. > 2) With GRIFFIN-194, DB list followed by number of table list operations can > be replaced by "list all table names in all databases" operation. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] incubator-griffin pull request #420: Fix doc issue.
Github user asfgit closed the pull request at: https://github.com/apache/incubator-griffin/pull/420 ---
[GitHub] incubator-griffin issue #420: Fix doc issue.
Github user bhlx3lyx7 commented on the issue: https://github.com/apache/incubator-griffin/pull/420 Thanks @toyboxman , we can merge this PR. ---
[jira] [Commented] (GRIFFIN-130) [Site] Modify griffin site template
[ https://issues.apache.org/jira/browse/GRIFFIN-130?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641262#comment-16641262 ] ASF GitHub Bot commented on GRIFFIN-130: Github user asfgit closed the pull request at: https://github.com/apache/incubator-griffin-site/pull/2 > [Site] Modify griffin site template > --- > > Key: GRIFFIN-130 > URL: https://issues.apache.org/jira/browse/GRIFFIN-130 > Project: Griffin (Incubating) > Issue Type: Task >Reporter: Lionel Liu >Assignee: Juan Li >Priority: Major > Original Estimate: 24h > Remaining Estimate: 24h > > Change the template of griffin site -- This message was sent by Atlassian JIRA (v7.6.3#76005)
[GitHub] incubator-griffin-site pull request #3: add user story for p13n
Github user asfgit closed the pull request at: https://github.com/apache/incubator-griffin-site/pull/3 ---
[GitHub] incubator-griffin-site pull request #1: modify plan.md
Github user asfgit closed the pull request at: https://github.com/apache/incubator-griffin-site/pull/1 ---
[GitHub] incubator-griffin-site pull request #10: update Nikolay as committer from co...
Github user asfgit closed the pull request at: https://github.com/apache/incubator-griffin-site/pull/10 ---
[GitHub] incubator-griffin-site pull request #2: griffin site - GRIFFIN-130 - GRIFFIN...
Github user asfgit closed the pull request at: https://github.com/apache/incubator-griffin-site/pull/2 ---
[jira] [Commented] (GRIFFIN-201) Hive metastore calls failing frequently
[
https://issues.apache.org/jira/browse/GRIFFIN-201?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16641223#comment-16641223
]
Nikolay Sokolov commented on GRIFFIN-201:
-
Provided fix as part of https://github.com/apache/incubator-griffin/pull/431
> Hive metastore calls failing frequently
> ---
>
> Key: GRIFFIN-201
> URL: https://issues.apache.org/jira/browse/GRIFFIN-201
> Project: Griffin (Incubating)
> Issue Type: Bug
>Reporter: Nikolay Sokolov
>Priority: Major
>
> Observed both with vanilla docker image, current trunk in docker and on
> real-world deployments.
> Typical symptoms:
> * exception: org.apache.thrift.TApplicationException get_all_tables failed:
> out of sequence response
> * Got exception: org.apache.thrift.transport.TTransportException
> java.io.IOException: Stream closed
> * o.a.t.t.TIOStreamTransport : Error closing output stream.
> {code:none}
> 2018-10-07 14:35:13.797 ERROR 6148 --- [nio-8080-exec-1] h.log
> : Got exception: org.apache.thrift.TApplicationException
> get_all_tables failed: out of sequence response
> org.apache.thrift.TApplicationException: get_all_tables failed: out of
> sequence response
> at
> org.apache.thrift.TServiceClient.receiveBase(TServiceClient.java:84)
> ~[libthrift-0.9.3.jar!/:0.9.3]
> at
> org.apache.hadoop.hive.metastore.api.ThriftHiveMetastore$Client.recv_get_all_tables(ThriftHiveMetastore.java:1443)
> ~[hive-metastore-2.2.0.jar!/:2.2.0]
> at
> org.apache.hadoop.hive.metastore.api.ThriftHiveMetastore$Client.get_all_tables(ThriftHiveMetastore.java:1430)
> ~[hive-metastore-2.2.0.jar!/:2.2.0]
> at
> org.apache.hadoop.hive.metastore.HiveMetaStoreClient.getAllTables(HiveMetaStoreClient.java:1425)
> [hive-metastore-2.2.0.jar!/:2.2.0]
> at
> org.apache.griffin.core.metastore.hive.HiveMetaStoreServiceImpl.getAllTableNames(HiveMetaStoreServiceImpl.java:87)
> [classes!/:0.3.1-incubating-SNAPSHOT]
> at
> org.apache.griffin.core.metastore.hive.HiveMetaStoreServiceImpl$$FastClassBySpringCGLIB$$d0fbb087.invoke()
> [classes!/:0.3.1-incubating-SNAPSHOT]
> at
> org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:204)
> [spring-core-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:721)
> [spring-aop-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:157)
> [spring-aop-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.springframework.cache.interceptor.CacheInterceptor$1.invoke(CacheInterceptor.java:52)
> [spring-context-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.springframework.cache.interceptor.CacheAspectSupport.invokeOperation(CacheAspectSupport.java:345)
> [spring-context-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.springframework.cache.interceptor.CacheAspectSupport.execute(CacheAspectSupport.java:408)
> [spring-context-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.springframework.cache.interceptor.CacheAspectSupport.execute(CacheAspectSupport.java:327)
> [spring-context-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.springframework.cache.interceptor.CacheInterceptor.invoke(CacheInterceptor.java:61)
> [spring-context-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:179)
> [spring-aop-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:656)
> [spring-aop-4.3.6.RELEASE.jar!/:4.3.6.RELEASE]
> at
> org.apache.griffin.core.metastore.hive.HiveMetaStoreServiceImpl$$EnhancerBySpringCGLIB$$390ab847.getAllTableNames()
> [classes!/:0.3.1-incubating-SNAPSHOT]
> at
> org.apache.griffin.core.metastore.hive.HiveMetaStoreController.getAllTableNames(HiveMetaStoreController.java:47)
> [classes!/:0.3.1-incubating-SNAPSHOT]
> {code}
> {code:none}
> 2018-10-07 14:35:13.848 ERROR 6148 --- [nio-8080-exec-3] h.log
> : Got exception:
> org.apache.thrift.transport.TTransportException java.io.IOException: Stream
> closed
> org.apache.thrift.transport.TTransportException: java.io.IOException: Stream
> closed
> at
> org.apache.thrift.transport.TIOStreamTransport.read(TIOStreamTransport.java:129)
> ~[libthrift-0.9.3.jar!/:0.9.3]
> at org.apache.thrift.transport.TTransport.readAll(TTransport.java:86)
> ~[libthrift-0.9.3.jar!/:0.9.3]
> at
> org.apache.thrift.protocol.TBinaryProtocol.readStringBody(TBinaryProtocol.java:380)
>
[GitHub] incubator-griffin-site pull request #10: update Nikolay as committer from co...
GitHub user guoyuepeng opened a pull request: https://github.com/apache/incubator-griffin-site/pull/10 update Nikolay as committer from contributor You can merge this pull request into a Git repository by running: $ git pull https://github.com/guoyuepeng/incubator-griffin-site master Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-griffin-site/pull/10.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #10 commit b0a5629abcc14402afa05e0c8fe585aefb35d044 Author: William Guo Date: 2018-10-07T20:46:13Z Update community.md commit acf784bd823e6386cbb267992efcbe4b614c0d93 Author: William Guo Date: 2018-10-07T20:49:13Z Update contributors.md commit 62bfac66359244dee50b63fe5009b46008fc3368 Author: William Guo Date: 2018-10-07T20:50:11Z Update community.md ---
[jira] [Commented] (GRIFFIN-195) [UI] Don't list all table objects from UI
[ https://issues.apache.org/jira/browse/GRIFFIN-195?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16640994#comment-16640994 ] ASF GitHub Bot commented on GRIFFIN-195: GitHub user chemikadze opened a pull request: https://github.com/apache/incubator-griffin/pull/431 [GRIFFIN-195] Don't list all tables from UI Initially, load only list of databases and list of table names. Table information is loaded only if user clicks on specific table. You can merge this pull request into a Git repository by running: $ git pull https://github.com/chemikadze/incubator-griffin GRIFFIN-193 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/incubator-griffin/pull/431.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #431 commit bca5bc78a316c435742df1b862f7c8f4b09e1458 Author: Nikolay Sokolov Date: 2018-10-07T00:39:07Z [GRIFFIN-195] Don't list all tables from UI Initially, load only list of databases and list of table names. Table information is loaded only if user clicks on specific table. commit 366427e747d6fa2aa2149efc9b79dab5559814d0 Author: Nikolay Sokolov Date: 2018-10-07T08:14:40Z Trigger tree rendering only when all dbs loaded > [UI] Don't list all table objects from UI > - > > Key: GRIFFIN-195 > URL: https://issues.apache.org/jira/browse/GRIFFIN-195 > Project: Griffin (Incubating) > Issue Type: Sub-task >Reporter: Nikolay Sokolov >Priority: Major > > Listing all table objects when rendering list of tables to profile takes lots > of time, and adds significant latency even if response is cached, when number > of tables is big and/or schema contains lots of columns. > Two solutions are possible: > 1) Without GRIFFIN-194, use DB list API and then list tables API is called > for each DB, in order to collect all tables. When table is clicked on, > request to get table details can be made. > 2) With GRIFFIN-194, DB list followed by number of table list operations can > be replaced by "list all table names in all databases" operation. -- This message was sent by Atlassian JIRA (v7.6.3#76005)
