[GitHub] [incubator-gobblin] asfgit closed pull request #2588: GOBBLIN-721: Remove additional ack. Simplify watermark manager (remov…

asfgit closed pull request #2588: GOBBLIN-721: Remove additional ack. Simplify watermark manager (remov… URL: https://github.com/apache/incubator-gobblin/pull/2588 This is an automated message from the Apache Git Service.

[GitHub] [incubator-gobblin] jhsenjaliya opened a new pull request #2586: [GOBBLIN-719] fix invalid git links for classes in docs

jhsenjaliya opened a new pull request #2586: [GOBBLIN-719] fix invalid git links for classes in docs URL: https://github.com/apache/incubator-gobblin/pull/2586 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all

[GitHub] [incubator-gobblin] ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r271429375 ## File path:

[GitHub] [incubator-gobblin] ibuenros commented on issue #2584: Filter Out Empty MultiWorkUnits

ibuenros commented on issue #2584: Filter Out Empty MultiWorkUnits URL: https://github.com/apache/incubator-gobblin/pull/2584#issuecomment-479122691 +1 This is an automated message from the Apache Git Service. To respond to

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r271550178 ## File path: gobblin-docs/user-guide/Gobblin-CLI.md ## @@

[GitHub] [incubator-gobblin] yukuai518 opened a new pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 opened a new pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I

[GitHub] [incubator-gobblin] asfgit closed pull request #2573: [GOBBLIN-703] Allow planning job to be running in non-blocking mode

asfgit closed pull request #2573: [GOBBLIN-703] Allow planning job to be running in non-blocking mode URL: https://github.com/apache/incubator-gobblin/pull/2573 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r268845649 ## File path: gobblin-docs/user-guide/Gobblin-CLI.md ## @@

[GitHub] [incubator-gobblin] yukuai518 commented on issue #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 commented on issue #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#issuecomment-476421024 @ibuenros can you help review?

[GitHub] [incubator-gobblin] sv2000 commented on issue #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution…

sv2000 commented on issue #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution… URL: https://github.com/apache/incubator-gobblin/pull/2580#issuecomment-476498221 @arjun4084346 @yukuai518 Can you please review this PR?

[GitHub] [incubator-gobblin] sv2000 opened a new pull request #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution…

sv2000 opened a new pull request #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution… URL: https://github.com/apache/incubator-gobblin/pull/2580 …s in Gobblin-as-a-Service. Dear Gobblin maintainers, Please accept this PR. I understand that it will not

[GitHub] [incubator-gobblin] sv2000 commented on issue #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution…

sv2000 commented on issue #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution… URL: https://github.com/apache/incubator-gobblin/pull/2580#issuecomment-477265883 @arjun4084346 addressed your comment.

[GitHub] [incubator-gobblin] asfgit closed pull request #2576: [GOBBLIN-706]enable dynamic mappers

asfgit closed pull request #2576: [GOBBLIN-706]enable dynamic mappers URL: https://github.com/apache/incubator-gobblin/pull/2576 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-gobblin] asfgit closed pull request #2584: Filter Out Empty MultiWorkUnits

asfgit closed pull request #2584: Filter Out Empty MultiWorkUnits URL: https://github.com/apache/incubator-gobblin/pull/2584 This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [incubator-gobblin] autumnust commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

autumnust commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r271922811 ## File path: conf/yarn/reference.conf ## @@ -38,6 +38,6 @@

[GitHub] [incubator-gobblin] autumnust commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

autumnust commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r271925108 ## File path: gobblin-docs/user-guide/Gobblin-CLI.md ## @@ -28,29

[GitHub] [incubator-gobblin] autumnust commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

autumnust commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r271923038 ## File path: conf/yarn/application.conf ## @@ -22,15 +22,18 @@

[GitHub] [incubator-gobblin] ibuenros commented on issue #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

ibuenros commented on issue #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#issuecomment-479582467 +1 This is an automated

[GitHub] [incubator-gobblin] yukuai518 commented on issue #2586: [GOBBLIN-719] fix invalid git links for classes in docs

yukuai518 commented on issue #2586: [GOBBLIN-719] fix invalid git links for classes in docs URL: https://github.com/apache/incubator-gobblin/pull/2586#issuecomment-479586004 +1 Thanks for fixing all the link issues!

[GitHub] [incubator-gobblin] arjun4084346 commented on a change in pull request #2589: [GOBBLIN-722] Unschedule gaas flow

arjun4084346 commented on a change in pull request #2589: [GOBBLIN-722] Unschedule gaas flow URL: https://github.com/apache/incubator-gobblin/pull/2589#discussion_r272010852 ## File path: gobblin-runtime/src/main/java/org/apache/gobblin/scheduler/JobScheduler.java ## @@

[GitHub] [incubator-gobblin] arjun4084346 opened a new pull request #2589: [GOBBLIN-722] Unschedule gaas flow

arjun4084346 opened a new pull request #2589: [GOBBLIN-722] Unschedule gaas flow URL: https://github.com/apache/incubator-gobblin/pull/2589 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!

[GitHub] [incubator-gobblin] asfgit closed pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

asfgit closed pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579 This is an automated message from the Apache Git

[GitHub] [incubator-gobblin] sv2000 commented on a change in pull request #2589: [GOBBLIN-722] Unschedule gaas flow

sv2000 commented on a change in pull request #2589: [GOBBLIN-722] Unschedule gaas flow URL: https://github.com/apache/incubator-gobblin/pull/2589#discussion_r272006054 ## File path: gobblin-runtime/src/main/java/org/apache/gobblin/scheduler/JobScheduler.java ## @@ -351,8

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r272026110 ## File path: conf/yarn/application.conf ## @@ -22,15 +22,18 @@

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r272025514 ## File path: conf/yarn/reference.conf ## @@ -38,6 +38,6 @@

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r272026110 ## File path: conf/yarn/application.conf ## @@ -22,15 +22,18 @@

[GitHub] [incubator-gobblin] zxcware commented on a change in pull request #2577: GOBBLIN-708: Create SqlDatasetDescriptor for JDBC-sourced datasets.

zxcware commented on a change in pull request #2577: GOBBLIN-708: Create SqlDatasetDescriptor for JDBC-sourced datasets. URL: https://github.com/apache/incubator-gobblin/pull/2577#discussion_r272260089 ## File path:

[GitHub] [incubator-gobblin] zxcware commented on a change in pull request #2577: GOBBLIN-708: Create SqlDatasetDescriptor for JDBC-sourced datasets.

zxcware commented on a change in pull request #2577: GOBBLIN-708: Create SqlDatasetDescriptor for JDBC-sourced datasets. URL: https://github.com/apache/incubator-gobblin/pull/2577#discussion_r272259273 ## File path:

[GitHub] [incubator-gobblin] zxcware commented on a change in pull request #2577: GOBBLIN-708: Create SqlDatasetDescriptor for JDBC-sourced datasets.

zxcware commented on a change in pull request #2577: GOBBLIN-708: Create SqlDatasetDescriptor for JDBC-sourced datasets. URL: https://github.com/apache/incubator-gobblin/pull/2577#discussion_r272257572 ## File path:

[GitHub] [incubator-gobblin] zxcware commented on a change in pull request #2577: GOBBLIN-708: Create SqlDatasetDescriptor for JDBC-sourced datasets.

zxcware commented on a change in pull request #2577: GOBBLIN-708: Create SqlDatasetDescriptor for JDBC-sourced datasets. URL: https://github.com/apache/incubator-gobblin/pull/2577#discussion_r272259071 ## File path:

[GitHub] [incubator-gobblin] asfgit closed pull request #2589: [GOBBLIN-722] Unschedule gaas flow

asfgit closed pull request #2589: [GOBBLIN-722] Unschedule gaas flow URL: https://github.com/apache/incubator-gobblin/pull/2589 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-gobblin] jhsenjaliya commented on issue #2586: [GOBBLIN-719] fix invalid git links for classes in docs

jhsenjaliya commented on issue #2586: [GOBBLIN-719] fix invalid git links for classes in docs URL: https://github.com/apache/incubator-gobblin/pull/2586#issuecomment-480150157 @yukuai518 , can you merge the changes, I need to update docs as part of #2578 and it will hekop to avoid

[GitHub] [incubator-gobblin] ibuenros commented on issue #2591: [GOBBLIN-724] Upgrade throttling server so waiting until tokens can be used is done…

ibuenros commented on issue #2591: [GOBBLIN-724] Upgrade throttling server so waiting until tokens can be used is done… URL: https://github.com/apache/incubator-gobblin/pull/2591#issuecomment-480102141 @htran1 can you review? I can go over the changes with you if necessary.

[GitHub] [incubator-gobblin] ibuenros opened a new pull request #2591: [GOBBLIN-724] Upgrade throttling server so waiting until tokens can be used is done…

ibuenros opened a new pull request #2591: [GOBBLIN-724] Upgrade throttling server so waiting until tokens can be used is done… URL: https://github.com/apache/incubator-gobblin/pull/2591 … by the client instead of the server. See GOBBLIN-724. Dear Gobblin maintainers, Please

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r272445741 ## File path: gobblin-docs/user-guide/Gobblin-CLI.md ## @@

[GitHub] [incubator-gobblin] yukuai518 commented on issue #2586: [GOBBLIN-719] fix invalid git links for classes in docs

yukuai518 commented on issue #2586: [GOBBLIN-719] fix invalid git links for classes in docs URL: https://github.com/apache/incubator-gobblin/pull/2586#issuecomment-480198001 I dont have previllege to merge. @Hung Tran please help Jay with his pr. Get

[GitHub] [incubator-gobblin] arjun4084346 opened a new pull request #2592: [GOBBLIN-725] Jobstatusstoretomysql

arjun4084346 opened a new pull request #2592: [GOBBLIN-725] Jobstatusstoretomysql URL: https://github.com/apache/incubator-gobblin/pull/2592 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2592: [GOBBLIN-725] add a mysql based job status retriever

jhsenjaliya commented on a change in pull request #2592: [GOBBLIN-725] add a mysql based job status retriever URL: https://github.com/apache/incubator-gobblin/pull/2592#discussion_r272788508 ## File path:

[GitHub] [incubator-gobblin] asfgit closed pull request #2583: [GOBBLIN-716] Add lineage in FileBasedSource

asfgit closed pull request #2583: [GOBBLIN-716] Add lineage in FileBasedSource URL: https://github.com/apache/incubator-gobblin/pull/2583 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-gobblin] ZihanLi58 opened a new pull request #2584: Zihli branch

ZihanLi58 opened a new pull request #2584: Zihli branch URL: https://github.com/apache/incubator-gobblin/pull/2584 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 - [ ] My

[GitHub] [incubator-gobblin] ZihanLi58 commented on issue #2584: Filter Out Empty MultiWorkUnits

ZihanLi58 commented on issue #2584: Filter Out Empty MultiWorkUnits URL: https://github.com/apache/incubator-gobblin/pull/2584#issuecomment-478145369 @ibuenros please take a look at this change This is an automated message

[GitHub] [incubator-gobblin] arjun4084346 opened a new pull request #2585: [GOBBLIN-718] Handle passing headers from slave to master

arjun4084346 opened a new pull request #2585: [GOBBLIN-718] Handle passing headers from slave to master URL: https://github.com/apache/incubator-gobblin/pull/2585 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all

[GitHub] [incubator-gobblin] sv2000 commented on issue #2585: [GOBBLIN-718] Handle passing headers from slave to master

sv2000 commented on issue #2585: [GOBBLIN-718] Handle passing headers from slave to master URL: https://github.com/apache/incubator-gobblin/pull/2585#issuecomment-478306895 Didn't we decide we are going to implement an option to unschedule flows, which will not delete state store? In

[GitHub] [incubator-gobblin] asfgit closed pull request #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution…

asfgit closed pull request #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution… URL: https://github.com/apache/incubator-gobblin/pull/2580 This is an automated message from the Apache Git Service.

[GitHub] [incubator-gobblin] ibuenros commented on a change in pull request #2582: UnitTest for KafkaSource

ibuenros commented on a change in pull request #2582: UnitTest for KafkaSource URL: https://github.com/apache/incubator-gobblin/pull/2582#discussion_r270197982 ## File path: gobblin-modules/gobblin-kafka-09/src/test/java/org/apache/gobblin/kafka/KafakaSourceTest.java ##

[GitHub] [incubator-gobblin] ibuenros commented on a change in pull request #2582: UnitTest for KafkaSource

ibuenros commented on a change in pull request #2582: UnitTest for KafkaSource URL: https://github.com/apache/incubator-gobblin/pull/2582#discussion_r270198707 ## File path: gobblin-modules/gobblin-kafka-09/src/test/java/org/apache/gobblin/kafka/KafakaSourceTest.java ##

[GitHub] [incubator-gobblin] yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269328551 ## File path:

[GitHub] [incubator-gobblin] yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269327266 ## File path:

[GitHub] [incubator-gobblin] yukuai518 opened a new pull request #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue.

yukuai518 opened a new pull request #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue. URL: https://github.com/apache/incubator-gobblin/pull/2581 Lazy load job specification from job catalog to avoid OOM issue. Dear Gobblin maintainers,

[GitHub] [incubator-gobblin] yukuai518 commented on issue #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue.

yukuai518 commented on issue #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue. URL: https://github.com/apache/incubator-gobblin/pull/2581#issuecomment-476867904 @htran1 Can you review?

[GitHub] [incubator-gobblin] yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269327266 ## File path:

[GitHub] [incubator-gobblin] ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269307236 ## File path:

[GitHub] [incubator-gobblin] ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269307676 ## File path:

[GitHub] [incubator-gobblin] ZihanLi58 commented on issue #2582: UnitTest for KafkaSource

ZihanLi58 commented on issue #2582: UnitTest for KafkaSource URL: https://github.com/apache/incubator-gobblin/pull/2582#issuecomment-477367718 @ibuenros @htran1 @autumnust This is an automated message from the Apache Git

[GitHub] [incubator-gobblin] ZihanLi58 opened a new pull request #2582: UnitTest for KafkaSource

ZihanLi58 opened a new pull request #2582: UnitTest for KafkaSource URL: https://github.com/apache/incubator-gobblin/pull/2582 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

[GitHub] [incubator-gobblin] asfgit closed pull request #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue.

asfgit closed pull request #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue. URL: https://github.com/apache/incubator-gobblin/pull/2581 This is an automated message from the Apache Git

[GitHub] [incubator-gobblin] zxcware opened a new pull request #2583: [GOBBLIN-716] Add lineage in FileBasedSource

zxcware opened a new pull request #2583: [GOBBLIN-716] Add lineage in FileBasedSource URL: https://github.com/apache/incubator-gobblin/pull/2583 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!

[GitHub] [incubator-gobblin] zxcware commented on issue #2583: [GOBBLIN-716] Add lineage in FileBasedSource

zxcware commented on issue #2583: [GOBBLIN-716] Add lineage in FileBasedSource URL: https://github.com/apache/incubator-gobblin/pull/2583#issuecomment-477406587 @ibuenros Please take a look This is an automated message from

[GitHub] [incubator-gobblin] sv2000 commented on a change in pull request #2587: [GOBBLIN-720 Always delete state store

sv2000 commented on a change in pull request #2587: [GOBBLIN-720 Always delete state store URL: https://github.com/apache/incubator-gobblin/pull/2587#discussion_r271001383 ## File path: gobblin-runtime/src/main/java/org/apache/gobblin/runtime/job_monitor/KafkaJobMonitor.java

[GitHub] [incubator-gobblin] ibuenros commented on issue #2582: UnitTest for KafkaSource

ibuenros commented on issue #2582: UnitTest for KafkaSource URL: https://github.com/apache/incubator-gobblin/pull/2582#issuecomment-478697677 @htran1 can you merge? This is an automated message from the Apache Git Service.

[GitHub] [incubator-gobblin] arjun4084346 opened a new pull request #2587: [GOBBLIN-720 Always delete state store

arjun4084346 opened a new pull request #2587: [GOBBLIN-720 Always delete state store URL: https://github.com/apache/incubator-gobblin/pull/2587 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!

[GitHub] [incubator-gobblin] sv2000 commented on a change in pull request #2587: [GOBBLIN-720 Always delete state store

sv2000 commented on a change in pull request #2587: [GOBBLIN-720 Always delete state store URL: https://github.com/apache/incubator-gobblin/pull/2587#discussion_r271001196 ## File path: gobblin-runtime/src/main/java/org/apache/gobblin/runtime/job_monitor/KafkaJobMonitor.java

[GitHub] [incubator-gobblin] ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r271003747 ## File path:

[GitHub] [incubator-gobblin] ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

ibuenros commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r271003600 ## File path:

[GitHub] [incubator-gobblin] shirshanka opened a new pull request #2588: GOBBLIN-721: Remove additional ack. Simplify watermark manager (remov…

shirshanka opened a new pull request #2588: GOBBLIN-721: Remove additional ack. Simplify watermark manager (remov… URL: https://github.com/apache/incubator-gobblin/pull/2588 …e multiwriter) Dear Gobblin maintainers, Please accept this PR. I understand that it will not be

[GitHub] [incubator-gobblin] arjun4084346 commented on a change in pull request #2592: [GOBBLIN-725] add a mysql based job status retriever

arjun4084346 commented on a change in pull request #2592: [GOBBLIN-725] add a mysql based job status retriever URL: https://github.com/apache/incubator-gobblin/pull/2592#discussion_r272854573 ## File path:

[GitHub] [incubator-gobblin] arjun4084346 commented on a change in pull request #2592: [GOBBLIN-725] add a mysql based job status retriever

arjun4084346 commented on a change in pull request #2592: [GOBBLIN-725] add a mysql based job status retriever URL: https://github.com/apache/incubator-gobblin/pull/2592#discussion_r272854535 ## File path:

[GitHub] [incubator-gobblin] yukuai518 commented on a change in pull request #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue.

yukuai518 commented on a change in pull request #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue. URL: https://github.com/apache/incubator-gobblin/pull/2581#discussion_r269345097 ## File path:

[GitHub] [incubator-gobblin] htran1 commented on a change in pull request #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue.

htran1 commented on a change in pull request #2581: [GOBBLIN-713] Lazy load job specification from job catalog to avoid OOM issue. URL: https://github.com/apache/incubator-gobblin/pull/2581#discussion_r269339933 ## File path:

[GitHub] [incubator-gobblin] yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269327266 ## File path:

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r268845649 ## File path: gobblin-docs/user-guide/Gobblin-CLI.md ## @@

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r268845649 ## File path: gobblin-docs/user-guide/Gobblin-CLI.md ## @@

[GitHub] [incubator-gobblin] autumnust commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command

autumnust commented on a change in pull request #2578: [GOBBLIN-707] rewrite gobblin script to combine all modes and command URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r268778696 ## File path: gobblin-docs/user-guide/Gobblin-CLI.md ## @@ -28,29

[GitHub] [incubator-gobblin] yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269328551 ## File path:

[GitHub] [incubator-gobblin] yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269373209 ## File path:

[GitHub] [incubator-gobblin] yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269373209 ## File path:

[GitHub] [incubator-gobblin] yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow

yukuai518 commented on a change in pull request #2579: [GOBBLIN-712] Add version strategy pickup for ConfigBasedDataset distcp workflow URL: https://github.com/apache/incubator-gobblin/pull/2579#discussion_r269328551 ## File path:

[GitHub] [incubator-gobblin] jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707]-rewrite gobblin script combining all gobblin modes and …

jhsenjaliya commented on a change in pull request #2578: [GOBBLIN-707]-rewrite gobblin script combining all gobblin modes and … URL: https://github.com/apache/incubator-gobblin/pull/2578#discussion_r268408431 ## File path: bin/gobblin.sh ## @@ -17,50 +17,410 @@ #

[GitHub] [incubator-gobblin] jhsenjaliya opened a new pull request #2578: [GOBBLIN-707]-rewrite gobblin script combining all gobblin modes and …

jhsenjaliya opened a new pull request #2578: [GOBBLIN-707]-rewrite gobblin script combining all gobblin modes and … URL: https://github.com/apache/incubator-gobblin/pull/2578 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have

[GitHub] [incubator-gobblin] arjun4084346 commented on a change in pull request #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution…

arjun4084346 commented on a change in pull request #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution… URL: https://github.com/apache/incubator-gobblin/pull/2580#discussion_r269407533 ## File path:

[GitHub] [incubator-gobblin] arjun4084346 commented on issue #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution…

arjun4084346 commented on issue #2580: GOBBLIN-709: Provide an option to disallow concurrent flow execution… URL: https://github.com/apache/incubator-gobblin/pull/2580#issuecomment-476979857 LGTM +1 This is an automated

[GitHub] htran1 commented on a change in pull request #2556: GOBBLIN-684: Ensure buffered messages are flushed before close() in K…

htran1 commented on a change in pull request #2556: GOBBLIN-684: Ensure buffered messages are flushed before close() in K… URL: https://github.com/apache/incubator-gobblin/pull/2556#discussion_r257847795 ## File path:

[GitHub] sv2000 commented on a change in pull request #2556: GOBBLIN-684: Ensure buffered messages are flushed before close() in K…

sv2000 commented on a change in pull request #2556: GOBBLIN-684: Ensure buffered messages are flushed before close() in K… URL: https://github.com/apache/incubator-gobblin/pull/2556#discussion_r257866178 ## File path:

[GitHub] sv2000 commented on a change in pull request #2556: GOBBLIN-684: Ensure buffered messages are flushed before close() in K…

sv2000 commented on a change in pull request #2556: GOBBLIN-684: Ensure buffered messages are flushed before close() in K… URL: https://github.com/apache/incubator-gobblin/pull/2556#discussion_r257866302 ## File path:

[GitHub] arjun4084346 opened a new pull request #2561: [GOBBLIN-689] catch unchecked exceptions in KafkaSource

arjun4084346 opened a new pull request #2561: [GOBBLIN-689] catch unchecked exceptions in KafkaSource URL: https://github.com/apache/incubator-gobblin/pull/2561 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all

[GitHub] asfgit closed pull request #2562: [GOBBLIN-690] Fix the planning job relaunch name match.

asfgit closed pull request #2562: [GOBBLIN-690] Fix the planning job relaunch name match. URL: https://github.com/apache/incubator-gobblin/pull/2562 This is an automated message from the Apache Git Service. To respond to

[GitHub] yukuai518 opened a new pull request #2562: [GOBBLIN-690] Fix the planning job relaunch name match.

yukuai518 opened a new pull request #2562: [GOBBLIN-690] Fix the planning job relaunch name match. URL: https://github.com/apache/incubator-gobblin/pull/2562 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the

[GitHub] ibuenros commented on a change in pull request #2563: [GOBBLIN-691] Make format-specific component pluggable in compaction

ibuenros commented on a change in pull request #2563: [GOBBLIN-691] Make format-specific component pluggable in compaction URL: https://github.com/apache/incubator-gobblin/pull/2563#discussion_r261352026 ## File path:

[GitHub] ibuenros commented on a change in pull request #2563: [GOBBLIN-691] Make format-specific component pluggable in compaction

ibuenros commented on a change in pull request #2563: [GOBBLIN-691] Make format-specific component pluggable in compaction URL: https://github.com/apache/incubator-gobblin/pull/2563#discussion_r261408097 ## File path:

[GitHub] asfgit closed pull request #2564: GOBBLIN-692: Add support to query last K flow executions in Gobblin-a…

asfgit closed pull request #2564: GOBBLIN-692: Add support to query last K flow executions in Gobblin-a… URL: https://github.com/apache/incubator-gobblin/pull/2564 This is an automated message from the Apache Git Service.

[GitHub] yukuai518 commented on issue #2563: [GOBBLIN-691] Make format-specific component pluggable in compaction

yukuai518 commented on issue #2563: [GOBBLIN-691] Make format-specific component pluggable in compaction URL: https://github.com/apache/incubator-gobblin/pull/2563#issuecomment-468507094 +1 LGTM. This is an automated

[GitHub] sv2000 commented on issue #2548: [GOBBLIN-677] - Allow early termination of Gobblin jobs based on a predicate on the job progress

sv2000 commented on issue #2548: [GOBBLIN-677] - Allow early termination of Gobblin jobs based on a predicate on the job progress URL: https://github.com/apache/incubator-gobblin/pull/2548#issuecomment-468547996 @ibuenros can you ensure that the travis build is successful before @htran1

[GitHub] htran1 commented on issue #2565: [GOBBLIN-693] Add ORC hive serde manager

htran1 commented on issue #2565: [GOBBLIN-693] Add ORC hive serde manager URL: https://github.com/apache/incubator-gobblin/pull/2565#issuecomment-468736489 @autumnust, please take a look. This is an automated message from

[GitHub] htran1 opened a new pull request #2565: [GOBBLIN-693] Add ORC hive serde manager

htran1 opened a new pull request #2565: [GOBBLIN-693] Add ORC hive serde manager URL: https://github.com/apache/incubator-gobblin/pull/2565 Dear Gobblin maintainers, Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!

[GitHub] ibuenros commented on a change in pull request #2565: [GOBBLIN-693] Add ORC hive serde manager

ibuenros commented on a change in pull request #2565: [GOBBLIN-693] Add ORC hive serde manager URL: https://github.com/apache/incubator-gobblin/pull/2565#discussion_r261696977 ## File path: gobblin-hive-registration/src/main/java/org/apache/gobblin/hive/HiveSerDeManager.java

[GitHub] [incubator-gobblin] asfgit closed pull request #2548: [GOBBLIN-677] - Allow early termination of Gobblin jobs based on a predicate on the job progress

asfgit closed pull request #2548: [GOBBLIN-677] - Allow early termination of Gobblin jobs based on a predicate on the job progress URL: https://github.com/apache/incubator-gobblin/pull/2548 This is an automated message from

[GitHub] asfgit closed pull request #2556: GOBBLIN-684: Ensure buffered messages are flushed before close() in K…

asfgit closed pull request #2556: GOBBLIN-684: Ensure buffered messages are flushed before close() in K… URL: https://github.com/apache/incubator-gobblin/pull/2556 This is an automated message from the Apache Git Service.

[GitHub] asfgit closed pull request #2539: [GOBBLIN-666] Data too long for column 'property_key'

asfgit closed pull request #2539: [GOBBLIN-666] Data too long for column 'property_key' URL: https://github.com/apache/incubator-gobblin/pull/2539 This is an automated message from the Apache Git Service. To respond to the

[GitHub] asfgit closed pull request #2538: [GOBBLIN-669] Configuration Properties Glossary section of Docs hard to read

asfgit closed pull request #2538: [GOBBLIN-669] Configuration Properties Glossary section of Docs hard to read URL: https://github.com/apache/incubator-gobblin/pull/2538 This is an automated message from the Apache Git

[GitHub] asfgit closed pull request #2554: GOBBLIN-682: Create a new constructor for DatasetCleanerJob.

asfgit closed pull request #2554: GOBBLIN-682: Create a new constructor for DatasetCleanerJob. URL: https://github.com/apache/incubator-gobblin/pull/2554 This is an automated message from the Apache Git Service. To respond

  1   2   3   4   5   6   7   8   9   10   >