You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2018/11/03 06:27:00 UTC

[jira] [Commented] (FLINK-10769) port InMemoryExternalCatalog to java

    [ https://issues.apache.org/jira/browse/FLINK-10769?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16673944#comment-16673944 ] 

ASF GitHub Bot commented on FLINK-10769:
----------------------------------------

bowenli86 opened a new pull request #7012: [FLINK-10769] port InMemoryExternalCatalog to java
URL: https://github.com/apache/flink/pull/7012
 
 
   ## What is the purpose of the change
   
   In the Flink-Hive integration design, we propose a new `FlinkInMemoryCatalog` (FLINK-10697) for production use, unlink `InMemoryExternalCatalog` which is only used for testing and dev work. `FlinkInMemoryCatalog` will share some part with the existing `InMemoryExternalCatalog`, thus we need to make changes to `InMemoryExternalCatalog`.
   
   As discussed with @twalthr , we are moving away from Scala to Java, and we should write all new code/feature in Java to minimize migration efforts. Therefore, we will port `InMemoryExternalCatalog` to java first. This PR  only port scala to java with NO feature or behavior change.
   
   This is a pre-requisite for FLINK-10697
   
   ## Brief change log
   
   port `InMemoryExternalCatalog` and `InMemoryExternalCatalogTest` to java
   
   ## Verifying this change
   
   This change is already covered by existing tests, such as *InMemoryExternalCatalog*.
   
   ## Does this pull request potentially affect one of the following parts:
   
   none
   
   ## Documentation
   
     - Does this pull request introduce a new feature? (no)
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> port InMemoryExternalCatalog to java
> ------------------------------------
>
>                 Key: FLINK-10769
>                 URL: https://issues.apache.org/jira/browse/FLINK-10769
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table API &amp; SQL
>            Reporter: Bowen Li
>            Assignee: Bowen Li
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.8.0
>
>
> In the Flink-Hive integration design, we propose a new FlinkInMemoryCatalog (FLINK-10697) for production use. FlinkInMemoryCatalog will share some part with the existing InMemoryExternalCatalog, thus we need to make changes to InMemoryExternalCatalog.
> As we are moving away from Scala to Java, we should write all new code/feature in Java. Therefore, we will port InMemoryExternalCatalog to java first without any feature or behavior change.
> This is a pre-requisite for FLINK-10697



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)