[jira] [Work logged] (GOBBLIN-854) update config reader in standalone mode

2020-03-09 Thread ASF GitHub Bot (Jira)


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

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

Author: ASF GitHub Bot
Created on: 10/Mar/20 05:09
Start Date: 10/Mar/20 05:09
Worklog Time Spent: 10m 
  Work Description: jhsenjaliya commented on issue #2710: [GOBBLIN-854] use 
typesafe config instead of java properties
URL: 
https://github.com/apache/incubator-gobblin/pull/2710#issuecomment-596905929
 
 
   @sv2000 , can u pls review this?
 

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:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 400561)
Time Spent: 20m  (was: 10m)

> update config reader in standalone mode
> ---
>
> Key: GOBBLIN-854
> URL: https://issues.apache.org/jira/browse/GOBBLIN-854
> Project: Apache Gobblin
>  Issue Type: Improvement
>Affects Versions: 0.14.0
>Reporter: Jay Sen
>Priority: Major
> Fix For: 0.15.0
>
>  Time Spent: 20m
>  Remaining Estimate: 0h
>
> standalone mode {{SchedulerDaemon}} uses java properties, this ticket is to 
> use TypeSafe Config instead to make config standardized across the modes and 
> also enable config to take benefits of TypeSafe functionalities.
> Also it takes 2 different config file as argument, one as default and another 
> as custom, we probably only need one property file.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Work logged] (GOBBLIN-830) update config key to define job launcher type

2020-03-09 Thread ASF GitHub Bot (Jira)


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

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

Author: ASF GitHub Bot
Created on: 10/Mar/20 05:08
Start Date: 10/Mar/20 05:08
Worklog Time Spent: 10m 
  Work Description: jhsenjaliya commented on issue #2691: [GOBBLIN-830] 
launcher.type to job.launcher.type
URL: 
https://github.com/apache/incubator-gobblin/pull/2691#issuecomment-596905586
 
 
   @arjun4084346, @sv2000 can u pls take a look? Thanks
 

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:
us...@infra.apache.org


Issue Time Tracking
---

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

> update config key to define job launcher type
> -
>
> Key: GOBBLIN-830
> URL: https://issues.apache.org/jira/browse/GOBBLIN-830
> Project: Apache Gobblin
>  Issue Type: Improvement
>Reporter: Jay Sen
>Priority: Minor
>  Time Spent: 1h 20m
>  Remaining Estimate: 0h
>
> config key JOB_LAUNCHER_TYPE_KEY = "launcher.type" should be set to 
> "job.launcher.type"



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [incubator-gobblin] jhsenjaliya commented on issue #2710: [GOBBLIN-854] use typesafe config instead of java properties

2020-03-09 Thread GitBox
jhsenjaliya commented on issue #2710: [GOBBLIN-854] use typesafe config instead 
of java properties
URL: 
https://github.com/apache/incubator-gobblin/pull/2710#issuecomment-596905929
 
 
   @sv2000 , can u pls review this?


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-gobblin] jhsenjaliya commented on issue #2691: [GOBBLIN-830] launcher.type to job.launcher.type

2020-03-09 Thread GitBox
jhsenjaliya commented on issue #2691: [GOBBLIN-830] launcher.type to 
job.launcher.type
URL: 
https://github.com/apache/incubator-gobblin/pull/2691#issuecomment-596905586
 
 
   @arjun4084346, @sv2000 can u pls take a look? Thanks


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Work logged] (GOBBLIN-1075) Add option to return latest failed flows

2020-03-09 Thread ASF GitHub Bot (Jira)


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

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

Author: ASF GitHub Bot
Created on: 10/Mar/20 00:26
Start Date: 10/Mar/20 00:26
Worklog Time Spent: 10m 
  Work Description: codecov-io commented on issue #2915: [GOBBLIN-1075] Add 
option to return latest failed flows
URL: 
https://github.com/apache/incubator-gobblin/pull/2915#issuecomment-596841705
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=h1)
 Report
   > Merging 
[#2915](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=desc)
 into 
[master](https://codecov.io/gh/apache/incubator-gobblin/commit/f3d75e392c554771e3c098cafb648ec55fa40ba7?src=pr&el=desc)
 will **decrease** coverage by `<.01%`.
   > The diff coverage is `9.09%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/graphs/tree.svg?width=650&token=4MgURJ0bGc&height=150&src=pr)](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master#2915  +/-   ##
   
   - Coverage 45.87%   45.87%   -0.01% 
   - Complexity 9190 9191   +1 
   
 Files  1934 1934  
 Lines 7293172948  +17 
 Branches   8046 8051   +5 
   
   + Hits  3345833465   +7 
   - Misses3640436414  +10 
 Partials   3069 3069
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=tree) 
| Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...rg/apache/gobblin/service/FlowExecutionClient.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi1mbG93LWNvbmZpZy1zZXJ2aWNlL2dvYmJsaW4tZmxvdy1jb25maWctc2VydmljZS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vc2VydmljZS9GbG93RXhlY3V0aW9uQ2xpZW50LmphdmE=)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[...org/apache/gobblin/service/FlowStatusResource.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi1mbG93LWNvbmZpZy1zZXJ2aWNlL2dvYmJsaW4tZmxvdy1jb25maWctc2VydmljZS1zZXJ2ZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vc2VydmljZS9GbG93U3RhdHVzUmVzb3VyY2UuamF2YQ==)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[.../apache/gobblin/service/FlowExecutionResource.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi1mbG93LWNvbmZpZy1zZXJ2aWNlL2dvYmJsaW4tZmxvdy1jb25maWctc2VydmljZS1zZXJ2ZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vc2VydmljZS9GbG93RXhlY3V0aW9uUmVzb3VyY2UuamF2YQ==)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/gobblin/service/FlowStatusClient.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi1mbG93LWNvbmZpZy1zZXJ2aWNlL2dvYmJsaW4tZmxvdy1jb25maWctc2VydmljZS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vc2VydmljZS9GbG93U3RhdHVzQ2xpZW50LmphdmE=)
 | `72.34% <100%> (ø)` | `7 <0> (ø)` | :arrow_down: |
   | 
[...obblin/service/monitoring/FlowStatusGenerator.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1ydW50aW1lL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NlcnZpY2UvbW9uaXRvcmluZy9GbG93U3RhdHVzR2VuZXJhdG9yLmphdmE=)
 | `48% <5.88%> (-21.7%)` | `11 <0> (ø)` | |
   | 
[...lin/restli/throttling/ZookeeperLeaderElection.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2UvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2Utc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3Jlc3RsaS90aHJvdHRsaW5nL1pvb2tlZXBlckxlYWRlckVsZWN0aW9uLmphdmE=)
 | `72.22% <0%> (+2.22%)` | `13% <0%> (ø)` | :arrow_down: |
   | 
[...in/java/org/apache/gobblin/cluster/HelixUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvSGVsaXhVdGlscy5qYXZh)
 | `41.73% <0%> (+3.47%)` | `15% <0%> (+1%)` | :arrow_up: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?sr

[GitHub] [incubator-gobblin] codecov-io commented on issue #2915: [GOBBLIN-1075] Add option to return latest failed flows

2020-03-09 Thread GitBox
codecov-io commented on issue #2915: [GOBBLIN-1075] Add option to return latest 
failed flows
URL: 
https://github.com/apache/incubator-gobblin/pull/2915#issuecomment-596841705
 
 
   # 
[Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=h1)
 Report
   > Merging 
[#2915](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=desc)
 into 
[master](https://codecov.io/gh/apache/incubator-gobblin/commit/f3d75e392c554771e3c098cafb648ec55fa40ba7?src=pr&el=desc)
 will **decrease** coverage by `<.01%`.
   > The diff coverage is `9.09%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/graphs/tree.svg?width=650&token=4MgURJ0bGc&height=150&src=pr)](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=tree)
   
   ```diff
   @@ Coverage Diff  @@
   ## master#2915  +/-   ##
   
   - Coverage 45.87%   45.87%   -0.01% 
   - Complexity 9190 9191   +1 
   
 Files  1934 1934  
 Lines 7293172948  +17 
 Branches   8046 8051   +5 
   
   + Hits  3345833465   +7 
   - Misses3640436414  +10 
 Partials   3069 3069
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=tree) 
| Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...rg/apache/gobblin/service/FlowExecutionClient.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi1mbG93LWNvbmZpZy1zZXJ2aWNlL2dvYmJsaW4tZmxvdy1jb25maWctc2VydmljZS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vc2VydmljZS9GbG93RXhlY3V0aW9uQ2xpZW50LmphdmE=)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[...org/apache/gobblin/service/FlowStatusResource.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi1mbG93LWNvbmZpZy1zZXJ2aWNlL2dvYmJsaW4tZmxvdy1jb25maWctc2VydmljZS1zZXJ2ZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vc2VydmljZS9GbG93U3RhdHVzUmVzb3VyY2UuamF2YQ==)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[.../apache/gobblin/service/FlowExecutionResource.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi1mbG93LWNvbmZpZy1zZXJ2aWNlL2dvYmJsaW4tZmxvdy1jb25maWctc2VydmljZS1zZXJ2ZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vc2VydmljZS9GbG93RXhlY3V0aW9uUmVzb3VyY2UuamF2YQ==)
 | `0% <0%> (ø)` | `0 <0> (ø)` | :arrow_down: |
   | 
[...a/org/apache/gobblin/service/FlowStatusClient.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi1mbG93LWNvbmZpZy1zZXJ2aWNlL2dvYmJsaW4tZmxvdy1jb25maWctc2VydmljZS1jbGllbnQvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2dvYmJsaW4vc2VydmljZS9GbG93U3RhdHVzQ2xpZW50LmphdmE=)
 | `72.34% <100%> (ø)` | `7 <0> (ø)` | :arrow_down: |
   | 
[...obblin/service/monitoring/FlowStatusGenerator.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1ydW50aW1lL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3NlcnZpY2UvbW9uaXRvcmluZy9GbG93U3RhdHVzR2VuZXJhdG9yLmphdmE=)
 | `48% <5.88%> (-21.7%)` | `11 <0> (ø)` | |
   | 
[...lin/restli/throttling/ZookeeperLeaderElection.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1yZXN0bGkvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2UvZ29iYmxpbi10aHJvdHRsaW5nLXNlcnZpY2Utc2VydmVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL3Jlc3RsaS90aHJvdHRsaW5nL1pvb2tlZXBlckxlYWRlckVsZWN0aW9uLmphdmE=)
 | `72.22% <0%> (+2.22%)` | `13% <0%> (ø)` | :arrow_down: |
   | 
[...in/java/org/apache/gobblin/cluster/HelixUtils.java](https://codecov.io/gh/apache/incubator-gobblin/pull/2915/diff?src=pr&el=tree#diff-Z29iYmxpbi1jbHVzdGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9nb2JibGluL2NsdXN0ZXIvSGVsaXhVdGlscy5qYXZh)
 | `41.73% <0%> (+3.47%)` | `15% <0%> (+1%)` | :arrow_up: |
   
   --
   
   [Continue to review full report at 
Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=continue).
   > **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
   > `Δ = absolute  (impact)`, `ø = not affected`, `? = missing data`
   > Powered by 
[Codecov](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=footer).
 Last update 
[f3d75e3...738dd73](https://codecov.io/gh/apache/incubator-gobblin/pull/2915?src=pr&el=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
   


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 co

[jira] [Work logged] (GOBBLIN-1075) Add option to return latest failed flows

2020-03-09 Thread ASF GitHub Bot (Jira)


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

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

Author: ASF GitHub Bot
Created on: 09/Mar/20 22:56
Start Date: 09/Mar/20 22:56
Worklog Time Spent: 10m 
  Work Description: arjun4084346 commented on issue #2915: [GOBBLIN-1075] 
Add option to return latest failed flows
URL: 
https://github.com/apache/incubator-gobblin/pull/2915#issuecomment-596817919
 
 
   We should also code it in an flexible way, so that now the filter supports a 
particular flow status(es) , and in future maybe a filter to support all GET 
flowConfigs with a particular config.
   The second point can be really helpful, when users(SREs) have to add/change 
one config to multiple flows. This has happened in past. cc @Pawan Reddy
 

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:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 400429)
Time Spent: 40m  (was: 0.5h)

> Add option to return latest failed flows
> 
>
> Key: GOBBLIN-1075
> URL: https://issues.apache.org/jira/browse/GOBBLIN-1075
> Project: Apache Gobblin
>  Issue Type: Improvement
>Reporter: Jack Moseley
>Priority: Major
>  Time Spent: 40m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [incubator-gobblin] arjun4084346 commented on issue #2915: [GOBBLIN-1075] Add option to return latest failed flows

2020-03-09 Thread GitBox
arjun4084346 commented on issue #2915: [GOBBLIN-1075] Add option to return 
latest failed flows
URL: 
https://github.com/apache/incubator-gobblin/pull/2915#issuecomment-596817919
 
 
   We should also code it in an flexible way, so that now the filter supports a 
particular flow status(es) , and in future maybe a filter to support all GET 
flowConfigs with a particular config.
   The second point can be really helpful, when users(SREs) have to add/change 
one config to multiple flows. This has happened in past. cc @Pawan Reddy


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Work logged] (GOBBLIN-1075) Add option to return latest failed flows

2020-03-09 Thread ASF GitHub Bot (Jira)


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

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

Author: ASF GitHub Bot
Created on: 09/Mar/20 22:51
Start Date: 09/Mar/20 22:51
Worklog Time Spent: 10m 
  Work Description: arjun4084346 commented on issue #2915: [GOBBLIN-1075] 
Add option to return latest failed flows
URL: 
https://github.com/apache/incubator-gobblin/pull/2915#issuecomment-596816593
 
 
   I think instead of giving an option to return "failed" flows, we should give 
option to return flows with ANY status. e.g. how many flows are in COMPILED 
state, how many are RUNNING and so on
 

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:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 400427)
Time Spent: 0.5h  (was: 20m)

> Add option to return latest failed flows
> 
>
> Key: GOBBLIN-1075
> URL: https://issues.apache.org/jira/browse/GOBBLIN-1075
> Project: Apache Gobblin
>  Issue Type: Improvement
>Reporter: Jack Moseley
>Priority: Major
>  Time Spent: 0.5h
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [incubator-gobblin] arjun4084346 commented on issue #2915: [GOBBLIN-1075] Add option to return latest failed flows

2020-03-09 Thread GitBox
arjun4084346 commented on issue #2915: [GOBBLIN-1075] Add option to return 
latest failed flows
URL: 
https://github.com/apache/incubator-gobblin/pull/2915#issuecomment-596816593
 
 
   I think instead of giving an option to return "failed" flows, we should give 
option to return flows with ANY status. e.g. how many flows are in COMPILED 
state, how many are RUNNING and so on


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Work logged] (GOBBLIN-1075) Add option to return latest failed flows

2020-03-09 Thread ASF GitHub Bot (Jira)


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

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

Author: ASF GitHub Bot
Created on: 09/Mar/20 22:09
Start Date: 09/Mar/20 22:09
Worklog Time Spent: 10m 
  Work Description: jack-moseley commented on pull request #2915: 
[GOBBLIN-1075] Add option to return latest failed flows
URL: https://github.com/apache/incubator-gobblin/pull/2915
 
 
   Dear Gobblin maintainers,
   
   Please accept this PR. I understand that it will not be reviewed until I 
have checked off all the steps below!
   
   
   ### JIRA
   - [x] My PR addresses the following [Gobblin 
JIRA](https://issues.apache.org/jira/browse/GOBBLIN/) issues and references 
them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"
   - https://issues.apache.org/jira/browse/GOBBLIN-1075
   
   
   ### Description
   - [x] Here are some details about my PR, including screenshots (if 
applicable):
   
   Add query boolean option that will only return failed flow 
statuses/executions.
   
   ### Tests
   - [x] My PR adds the following unit tests __OR__ does not need testing for 
this extremely good reason:
   
   Updated test
   
   ### Commits
   - [x] My commits all reference JIRA issues in their subject lines, and I 
have squashed multiple commits if they address the same issue. In addition, my 
commits follow the guidelines from "[How to write a good git commit 
message](http://chris.beams.io/posts/git-commit/)":
   1. Subject is separated from body by a blank line
   2. Subject is limited to 50 characters
   3. Subject does not end with a period
   4. Subject uses the imperative mood ("add", not "adding")
   5. Body wraps at 72 characters
   6. Body explains "what" and "why", not "how"
   
   
 

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:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 400394)
Remaining Estimate: 0h
Time Spent: 10m

> Add option to return latest failed flows
> 
>
> Key: GOBBLIN-1075
> URL: https://issues.apache.org/jira/browse/GOBBLIN-1075
> Project: Apache Gobblin
>  Issue Type: Improvement
>Reporter: Jack Moseley
>Priority: Major
>  Time Spent: 10m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Work logged] (GOBBLIN-1075) Add option to return latest failed flows

2020-03-09 Thread ASF GitHub Bot (Jira)


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

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

Author: ASF GitHub Bot
Created on: 09/Mar/20 22:09
Start Date: 09/Mar/20 22:09
Worklog Time Spent: 10m 
  Work Description: jack-moseley commented on issue #2915: [GOBBLIN-1075] 
Add option to return latest failed flows
URL: 
https://github.com/apache/incubator-gobblin/pull/2915#issuecomment-596803339
 
 
   @arjun4084346 please review
 

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:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 400395)
Time Spent: 20m  (was: 10m)

> Add option to return latest failed flows
> 
>
> Key: GOBBLIN-1075
> URL: https://issues.apache.org/jira/browse/GOBBLIN-1075
> Project: Apache Gobblin
>  Issue Type: Improvement
>Reporter: Jack Moseley
>Priority: Major
>  Time Spent: 20m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [incubator-gobblin] jack-moseley opened a new pull request #2915: [GOBBLIN-1075] Add option to return latest failed flows

2020-03-09 Thread GitBox
jack-moseley opened a new pull request #2915: [GOBBLIN-1075] Add option to 
return latest failed flows
URL: https://github.com/apache/incubator-gobblin/pull/2915
 
 
   Dear Gobblin maintainers,
   
   Please accept this PR. I understand that it will not be reviewed until I 
have checked off all the steps below!
   
   
   ### JIRA
   - [x] My PR addresses the following [Gobblin 
JIRA](https://issues.apache.org/jira/browse/GOBBLIN/) issues and references 
them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"
   - https://issues.apache.org/jira/browse/GOBBLIN-1075
   
   
   ### Description
   - [x] Here are some details about my PR, including screenshots (if 
applicable):
   
   Add query boolean option that will only return failed flow 
statuses/executions.
   
   ### Tests
   - [x] My PR adds the following unit tests __OR__ does not need testing for 
this extremely good reason:
   
   Updated test
   
   ### Commits
   - [x] My commits all reference JIRA issues in their subject lines, and I 
have squashed multiple commits if they address the same issue. In addition, my 
commits follow the guidelines from "[How to write a good git commit 
message](http://chris.beams.io/posts/git-commit/)":
   1. Subject is separated from body by a blank line
   2. Subject is limited to 50 characters
   3. Subject does not end with a period
   4. Subject uses the imperative mood ("add", not "adding")
   5. Body wraps at 72 characters
   6. Body explains "what" and "why", not "how"
   
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-gobblin] jack-moseley commented on issue #2915: [GOBBLIN-1075] Add option to return latest failed flows

2020-03-09 Thread GitBox
jack-moseley commented on issue #2915: [GOBBLIN-1075] Add option to return 
latest failed flows
URL: 
https://github.com/apache/incubator-gobblin/pull/2915#issuecomment-596803339
 
 
   @arjun4084346 please review


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:
us...@infra.apache.org


With regards,
Apache Git Services


[jira] [Created] (GOBBLIN-1075) Add option to return latest failed flows

2020-03-09 Thread Jack Moseley (Jira)
Jack Moseley created GOBBLIN-1075:
-

 Summary: Add option to return latest failed flows
 Key: GOBBLIN-1075
 URL: https://issues.apache.org/jira/browse/GOBBLIN-1075
 Project: Apache Gobblin
  Issue Type: Improvement
Reporter: Jack Moseley






--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Work logged] (GOBBLIN-1072) Being more conservative on releasing YARN containers

2020-03-09 Thread ASF GitHub Bot (Jira)


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

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

Author: ASF GitHub Bot
Created on: 09/Mar/20 16:42
Start Date: 09/Mar/20 16:42
Worklog Time Spent: 10m 
  Work Description: asfgit commented on pull request #2912: [GOBBLIN-1072] 
Being more conservative on releasing containers
URL: https://github.com/apache/incubator-gobblin/pull/2912
 
 
   
 

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:
us...@infra.apache.org


Issue Time Tracking
---

Worklog Id: (was: 400225)
Time Spent: 2h 50m  (was: 2h 40m)

> Being more conservative on releasing YARN containers
> 
>
> Key: GOBBLIN-1072
> URL: https://issues.apache.org/jira/browse/GOBBLIN-1072
> Project: Apache Gobblin
>  Issue Type: Improvement
>Reporter: Lei Sun
>Priority: Major
> Fix For: 0.15.0
>
>  Time Spent: 2h 50m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Resolved] (GOBBLIN-1072) Being more conservative on releasing YARN containers

2020-03-09 Thread Lei Sun (Jira)


 [ 
https://issues.apache.org/jira/browse/GOBBLIN-1072?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Lei Sun resolved GOBBLIN-1072.
--
Fix Version/s: 0.15.0
   Resolution: Fixed

Issue resolved by pull request #2912
[https://github.com/apache/incubator-gobblin/pull/2912]

> Being more conservative on releasing YARN containers
> 
>
> Key: GOBBLIN-1072
> URL: https://issues.apache.org/jira/browse/GOBBLIN-1072
> Project: Apache Gobblin
>  Issue Type: Improvement
>Reporter: Lei Sun
>Priority: Major
> Fix For: 0.15.0
>
>  Time Spent: 2h 50m
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[GitHub] [incubator-gobblin] asfgit closed pull request #2912: [GOBBLIN-1072] Being more conservative on releasing containers

2020-03-09 Thread GitBox
asfgit closed pull request #2912: [GOBBLIN-1072] Being more conservative on 
releasing containers
URL: https://github.com/apache/incubator-gobblin/pull/2912
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services