[jira] [Work logged] (GOBBLIN-854) update config reader in standalone mode
[ 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
[ 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
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
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
[ 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
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
[ 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
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
[ 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
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
[ 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
[ 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
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
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
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
[ 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
[ 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
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