gobblin-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Work logged] (GOBBLIN-785) remove wrapper isPartition function, use table.isPartitioned instead
Date Wed, 19 Jun 2019 05:36:00 GMT

     [ https://issues.apache.org/jira/browse/GOBBLIN-785?focusedWorklogId=262788&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-262788
]

ASF GitHub Bot logged work on GOBBLIN-785:
------------------------------------------

                Author: ASF GitHub Bot
            Created on: 19/Jun/19 05:35
            Start Date: 19/Jun/19 05:35
    Worklog Time Spent: 10m 
      Work Description: shirshanka commented on issue #2650: [GOBBLIN-785] remove wrapper
isPartition function, use table.isPartitioned instead
URL: https://github.com/apache/incubator-gobblin/pull/2650#issuecomment-503413383
 
 
   Looks like all review comments have been addressed. 
   @ibuenros : can you re-check? 
   LGTM!
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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


Issue Time Tracking
-------------------

    Worklog Id:     (was: 262788)
    Time Spent: 1h 10m  (was: 1h)

> remove wrapper isPartition function, use table.isPartitioned instead
> --------------------------------------------------------------------
>
>                 Key: GOBBLIN-785
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-785
>             Project: Apache Gobblin
>          Issue Type: Improvement
>            Reporter: Jay Sen
>            Priority: Minor
>          Time Spent: 1h 10m
>  Remaining Estimate: 0h
>
> some places use {{HiveUtils.isPartitioned()}} which is un-necessary wrapper function,
and some places uses direct {{HiveTable.isPartitioned()}} the later should be used consistently.



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

Mime
View raw message