lucperkins opened a new pull request #2820: Fix website for GitHub Pages
URL: https://github.com/apache/incubator-heron/pull/2820
At the moment, the GitHub Pages site for the post-Apache-transition version
of Heron is largely broken because the root URL used to generate the assets is
for t
jerrypeng closed pull request #2820: Fix website for GitHub Pages
URL: https://github.com/apache/incubator-heron/pull/2820
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull
ashvina opened a new pull request #2821: Update Dhalion dependency version
URL: https://github.com/apache/incubator-heron/pull/2821
Dhalion version 0.2.x has many api and feature improvements. For e.g.
1. Deep-nested hashmaps have been replaced with filter-friendly table
structures.
lucperkins commented on issue #2820: Fix website for GitHub Pages
URL: https://github.com/apache/incubator-heron/pull/2820#issuecomment-377375846
@srkukarni At the moment, *none* of the links work because the site was
generated with a different base URL. Those links will all be broken until
joshfischer1108 commented on issue #2820: Fix website for GitHub Pages
URL: https://github.com/apache/incubator-heron/pull/2820#issuecomment-377376187
https://user-images.githubusercontent.com/5785700/38114218-2c3e2d60-336d-11e8-9598-1f93e381578f.png";>
-
joshfischer1108 commented on issue #2820: Fix website for GitHub Pages
URL: https://github.com/apache/incubator-heron/pull/2820#issuecomment-377376335
that is what I see when I hit the docs from the heronstreaming.io link
lucperkins commented on issue #2820: Fix website for GitHub Pages
URL: https://github.com/apache/incubator-heron/pull/2820#issuecomment-377377280
@joshfischer1108 Yes, this is a known issue. I'm updating the site as fast
as I can.
---
nwangtw opened a new pull request #2822: Add rate limit change log in stream
manager
URL: https://github.com/apache/incubator-heron/pull/2822
This is an automated message from the Apache Git Service.
To respond to the messag
lucperkins opened a new pull request #2823: Update Git submodule URL for GitHub
pages branch
URL: https://github.com/apache/incubator-heron/pull/2823
This is an automated message from the Apache Git Service.
To respond to th
lucperkins closed pull request #2823: Update Git submodule URL for GitHub pages
branch
URL: https://github.com/apache/incubator-heron/pull/2823
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As t
lucperkins opened a new pull request #2824: More website fixes
URL: https://github.com/apache/incubator-heron/pull/2824
This PR addresses some minor issues with the website's static asset pipeline
This is an automated message
lucperkins closed pull request #2824: More website fixes
URL: https://github.com/apache/incubator-heron/pull/2824
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (
billonahill commented on issue #2133: Migrate code to Apache github
URL:
https://github.com/apache/incubator-heron/issues/2133#issuecomment-377400380
Done.
This is an automated message from the Apache Git Service.
To respond
billonahill closed issue #2133: Migrate code to Apache github
URL: https://github.com/apache/incubator-heron/issues/2133
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHu
lucperkins opened a new pull request #2825: Fix URL issues associated with Hugo
upgrade
URL: https://github.com/apache/incubator-heron/pull/2825
This is an automated message from the Apache Git Service.
To respond to the mes
lucperkins closed pull request #2825: Fix URL issues associated with Hugo
upgrade
URL: https://github.com/apache/incubator-heron/pull/2825
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this i
erenavsarogullari opened a new pull request #2826: [WIP] [Streamlet Scala API]
Add Scala Streamlet Integration Tests Part I
URL: https://github.com/apache/incubator-heron/pull/2826
**Note:** This PR is currently work-in-progress to verify new Scala
Streamlet Integration Tests on Travis CI.
kramasamy commented on a change in pull request #2821: Update Dhalion
dependency version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178218286
##
File path: heron/executor/src/python/heron_executor.py
##
@@ -498,7 +498,7 @@ def _get_healthmgr_cmd(s
nwangtw opened a new issue #2827: Fix CI job for compiling/unit
tests/integration tests
URL: https://github.com/apache/incubator-heron/issues/2827
After migrating to apache, the old travis job stops working. Need to figure
out the new way to run CI.
---
nwangtw closed pull request #2822: Add rate limit change log in stream manager
URL: https://github.com/apache/incubator-heron/pull/2822
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a
nwangtw opened a new issue #2829: Update license to apache incubator
URL: https://github.com/apache/incubator-heron/issues/2829
This is an automated message from the Apache Git Service.
To respond to the message, please log o
nwangtw opened a new issue #2828: Rename package name from com.twitter.heron to
com.apache.com
URL: https://github.com/apache/incubator-heron/issues/2828
This is an automated message from the Apache Git Service.
To respond t
ashvina commented on a change in pull request #2821: Update Dhalion dependency
version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178357054
##
File path: heron/executor/src/python/heron_executor.py
##
@@ -498,7 +498,7 @@ def _get_healthmgr_cmd(sel
huijunwu commented on a change in pull request #2821: Update Dhalion dependency
version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178396799
##
File path:
heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/GrowingWaitQueueDetector.java
###
huijunwu commented on a change in pull request #2821: Update Dhalion dependency
version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178397892
##
File path:
heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/GrowingWaitQueueDetector.java
###
huijunwu commented on a change in pull request #2821: Update Dhalion dependency
version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178398628
##
File path:
heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/SkewDetector.java
##
@@ -15,
huijunwu commented on a change in pull request #2821: Update Dhalion dependency
version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178394376
##
File path: heron/executor/src/python/heron_executor.py
##
@@ -498,7 +498,7 @@ def _get_healthmgr_cmd(se
huijunwu commented on a change in pull request #2821: Update Dhalion dependency
version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178399304
##
File path:
heron/healthmgr/src/java/com/twitter/heron/healthmgr/diagnosers/SlowInstanceDiagnoser.java
#
huijunwu commented on a change in pull request #2821: Update Dhalion dependency
version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178395687
##
File path:
heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/BackPressureDetector.java
##
ashvina commented on a change in pull request #2821: Update Dhalion dependency
version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r178407367
##
File path:
heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/BackPressureDetector.java
##
joshfischer1108 opened a new pull request #2830: removing abstract requirement
URL: https://github.com/apache/incubator-heron/pull/2830
@kramasamy @nwangtw This is fix for ECO custom grouping definitions.
This is an automated
kramasamy closed pull request #2819: [Documentation] Improve Java Streamlet API
doc
URL: https://github.com/apache/incubator-heron/pull/2819
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this
kramasamy commented on issue #2092: Remove binary packages from heron repo
URL:
https://github.com/apache/incubator-heron/issues/2092#issuecomment-377698026
@billonahill - protobuf is already out.
This is an automated message
ajorgensen opened a new pull request #2831: Fix bug in dry-run argument for
update command
URL: https://github.com/apache/incubator-heron/pull/2831
Since list_extra_args is an array the `+=` operator deconstructs a
string and add each character as an individual array element. This made
jerrypeng opened a new pull request #2832: allow storm topologies to use the
heron state interface
URL: https://github.com/apache/incubator-heron/pull/2832
To use the heron state interface uses will need to add these configs:
conf.put(com.twitter.heron.api.Config.TOPOLOGY_RELIABILITY
jerrypeng commented on issue #2832: allow storm topologies to use the heron
state interface
URL: https://github.com/apache/incubator-heron/pull/2832#issuecomment-377726381
@kramasamy
This is an automated message from the Apa
kramasamy commented on issue #2832: allow storm topologies to use the heron
state interface
URL: https://github.com/apache/incubator-heron/pull/2832#issuecomment-377726795
Ok. Can we have the two configs at heron also available in storm namespace -
users in storm namespace should not use h
jerrypeng commented on issue #2832: allow storm topologies to use the heron
state interface
URL: https://github.com/apache/incubator-heron/pull/2832#issuecomment-377728057
The configs don’t make sense for storm though
On Sat, Mar 31, 2018 at 3:07 PM Karthik Ramasamy
wrote:
ashvina commented on a change in pull request #2821: Update Dhalion dependency
version
URL: https://github.com/apache/incubator-heron/pull/2821#discussion_r17857
##
File path:
heron/healthmgr/src/java/com/twitter/heron/healthmgr/detectors/GrowingWaitQueueDetector.java
kramasamy commented on issue #2832: allow storm topologies to use the heron
state interface
URL: https://github.com/apache/incubator-heron/pull/2832#issuecomment-377732239
It feels weird to use Heron namespace config - which is always needed for
stateful topologies.
The second call
kramasamy commented on issue #2831: Fix bug in dry-run argument for update
command
URL: https://github.com/apache/incubator-heron/pull/2831#issuecomment-377732285
+1
This is an automated message from the Apache Git Service.
kramasamy closed pull request #2830: removing abstract requirement
URL: https://github.com/apache/incubator-heron/pull/2830
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull
kramasamy closed pull request #2831: Fix bug in dry-run argument for update
command
URL: https://github.com/apache/incubator-heron/pull/2831
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this
kramasamy opened a new pull request #2833: removed
glog/gperftools/libevent/libunwind/zookeeper tar.gz from source
URL: https://github.com/apache/incubator-heron/pull/2833
Instead all of them are downloaded on the fly and compiled
WORKSPACE file consists of one entry for each software it
jerrypeng commented on issue #2832: allow storm topologies to use the heron
state interface
URL: https://github.com/apache/incubator-heron/pull/2832#issuecomment-377736243
@kramasamy this is kinda of a hack to begin with. Storm doesn't have
exactly once state. If users want exactly once
sreev opened a new issue #2834: update README.md
URL: https://github.com/apache/incubator-heron/issues/2834
fix travis links
add meetup group link
This is an automated message from the Apache Git Service.
To respond to the
sreev opened a new pull request #2835: update README.md
URL: https://github.com/apache/incubator-heron/pull/2835
fix travis links
add meetup group link
https://github.com/apache/incubator-heron/issues/2834
This is an
sreev opened a new issue #2836: mac xcode upgrades fails build
URL: https://github.com/apache/incubator-heron/issues/2836
`
$ bazel build --config=darwin heron/...
Extracting Bazel installation...
...
ERROR:
/private/var/tmp/_bazel_svaddi/ad590ad4eafb86a9912ced0bf3547c8f
sreev commented on issue #2836: mac xcode upgrades fails build
URL:
https://github.com/apache/incubator-heron/issues/2836#issuecomment-377755548
Troubleshoot:
$ cat $(bazel info output_base)/external/local_config_xcode/BUILD
package(default_visibility = ['//visibility:public'])
sreev commented on issue #2836: mac xcode upgrades fails build
URL:
https://github.com/apache/incubator-heron/issues/2836#issuecomment-377755548
Troubleshoot:
> $ cat $(bazel info output_base)/external/local_config_xcode/BUILD
package(default_visibility = ['//visibility:public'])
sreev commented on issue #2836: mac xcode upgrades fails build
URL:
https://github.com/apache/incubator-heron/issues/2836#issuecomment-377755548
Troubleshoot:
$ cat $(bazel info output_base)/external/local_config_xcode/BUILD
package(default_visibility = ['//visibility:public'])
sreev commented on issue #2836: mac xcode upgrades fails build
URL:
https://github.com/apache/incubator-heron/issues/2836#issuecomment-377755548
Troubleshoot:
>
$ cat $(bazel info output_base)/external/local_config_xcode/BUILD
package(default_visibility = ['//visibility:public
sreev commented on issue #2836: mac xcode upgrades fails build
URL:
https://github.com/apache/incubator-heron/issues/2836#issuecomment-377755548
Troubleshoot:
> $ cat $(bazel info output_base)/external/local_config_xcode/BUILD
> package(default_visibility = ['//visibility:public']
sreev commented on issue #2836: mac xcode upgrades fails build
URL:
https://github.com/apache/incubator-heron/issues/2836#issuecomment-377755548
Troubleshoot:
> $ cat $(bazel info output_base)/external/local_config_xcode/BUILD
> package(default_visibility = ['//visibility:public']
sreev commented on issue #2836: mac xcode upgrades fails build
URL:
https://github.com/apache/incubator-heron/issues/2836#issuecomment-377755683
Fix:
> $ bazel clean --expunge
> $ sudo xcode-select -s /Applications/Xcode.app/Contents/Developer
> $ sudo xcodebuild -license
>
sreev commented on issue #2836: mac xcode upgrades fails build
URL:
https://github.com/apache/incubator-heron/issues/2836#issuecomment-377755860
Add to Documentation, before closing the issue.
This is an automated message fro
kramasamy closed pull request #2835: update README.md
URL: https://github.com/apache/incubator-heron/pull/2835
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (fro
kramasamy opened a new issue #2837: Consolidate test data paths into simple
constant
URL: https://github.com/apache/incubator-heron/issues/2837
Such as in the file
heron/uploaders/tests/java/com/twitter/heron/uploader/localfs/LocalFileSystemConstantsTest.java
To identify all
kramasamy commented on issue #2832: allow storm topologies to use the heron
state interface
URL: https://github.com/apache/incubator-heron/pull/2832#issuecomment-377768234
@jerrypeng - my key argument the user needs to be aware that a configuration
from Heron API required for using with St
srkukarni commented on issue #2832: allow storm topologies to use the heron
state interface
URL: https://github.com/apache/incubator-heron/pull/2832#issuecomment-377793771
@jerrypeng I'm not sure if this is the right approach to take. You are
breaking storm compatibility as well as breakin
erenavsarogullari opened a new pull request #2838: [Documentation] Improve
JavaDoc and Documentation
URL: https://github.com/apache/incubator-heron/pull/2838
This PR aims the following changes:
- Fixes for `javadocs`
- Fixes for Documentation
-
huijunw opened a new pull request #2839: [WIP] fix healthmgr metrics
URL: https://github.com/apache/incubator-heron/pull/2839
after #2821
This PR fix the metrics reporting for healthmgr
This is an automated message from
kramasamy closed pull request #2833: removed
glog/gperftools/libevent/libunwind/zookeeper tar.gz from source
URL: https://github.com/apache/incubator-heron/pull/2833
This is an automated message from the Apache Git Service.
kramasamy commented on issue #2092: Remove binary packages from heron repo
URL:
https://github.com/apache/incubator-heron/issues/2092#issuecomment-377855627
This is addressed by the PR #2833 - let me know if anything else is remaining
---
kramasamy closed issue #2092: Remove binary packages from heron repo
URL: https://github.com/apache/incubator-heron/issues/2092
This is an automated message from the Apache Git Service.
To respond to the message, please log o
kramasamy commented on issue #2788: Remove Scribe support?
URL:
https://github.com/apache/incubator-heron/issues/2788#issuecomment-37783
@bjonnh - can you try this with 0.17.8 and see if this works?
This is an automated m
sreev opened a new pull request #2840: merge rename from com.twitter to
org.apache [apache#2828]
URL: https://github.com/apache/incubator-heron/pull/2840
rename folders
rename imports
update script links
update doc links
sreev commented on issue #2840: merge rename from com.twitter to org.apache
[apache#2828]
URL: https://github.com/apache/incubator-heron/pull/2840#issuecomment-377868011
> (06:17:20) ERROR:
/home/travis/build/apache/incubator-heron/heron/statemgrs/src/java/BUILD:82:1:
Executing extra_acti
sreev commented on issue #2840: merge rename from com.twitter to org.apache
[apache#2828]
URL: https://github.com/apache/incubator-heron/pull/2840#issuecomment-377881008
> (08:08:08) ERROR:
/home/travis/build/apache/incubator-heron/heron/uploaders/tests/java/BUILD:58:1:
Executing extra_ac
sreev opened a new issue #2841: remove com.sun.* package dependency
URL: https://github.com/apache/incubator-heron/issues/2841
- AbstractWebSink.java
This is an automated message from the Apache Git Service.
To respond to
sreev commented on issue #2840: merge rename from com.twitter to org.apache
[apache#2828]
URL: https://github.com/apache/incubator-heron/pull/2840#issuecomment-377905282
> [ERROR]
/home/travis/.cache/bazel/_bazel_travis/be6dac4936703c7eedcb4f5cf38cdd65/execroot/org_apache_heron/heron/uploa
kramasamy closed pull request #2838: [Documentation] Improve JavaDocs and
Documentation
URL: https://github.com/apache/incubator-heron/pull/2838
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As
bjonnh commented on issue #2788: Remove Scribe support?
URL:
https://github.com/apache/incubator-heron/issues/2788#issuecomment-377968830
Tried with latest master:
bazel build --config=ubuntu heron/...
INFO: Analysed 574 targets (288 packages loaded).
INFO: Found 574 targets...
ajorgensen opened a new pull request #2842: Add json format for dry-run
URL: https://github.com/apache/incubator-heron/pull/2842
This adds json as an optional format for the dry-run output. Json is
more easily machine readable than the other output formats and can be
used to make deplo
srkukarni commented on issue #2798: While emitting in spout, adhere to the
batch size limit
URL: https://github.com/apache/incubator-heron/pull/2798#issuecomment-378064627
retest this please
This is an automated message from
sreev commented on issue #2840: merge rename from com.twitter to org.apache
[apache#2828]
URL: https://github.com/apache/incubator-heron/pull/2840#issuecomment-378066765
> Making install in doc
> make[1]: Entering directory
`/home/travis/.cache/bazel/_bazel_travis/be6dac4936703c7eedcb4f
sreev commented on issue #2840: merge rename from com.twitter to org.apache
[apache#2828]
URL: https://github.com/apache/incubator-heron/pull/2840#issuecomment-378066765
> Making install in doc
> make[1]: Entering directory
`/home/travis/.cache/bazel/_bazel_travis/be6dac4936703c7eedcb4f
sreev opened a new issue #2843: reduce build time
URL: https://github.com/apache/incubator-heron/issues/2843
install bazel the recommended way,
using apt-get instead install.sh script.
This is an automated message from
sreev commented on issue #2843: reduce build time
URL:
https://github.com/apache/incubator-heron/issues/2843#issuecomment-378085793
https://github.com/apache/incubator-heron/pull/2840/commits/33dcdcc83bd686a44d01bc24f71d8841e1cbbca9
https://github.com/apache/incubator-heron/pull/28
sreev commented on issue #2843: reduce build time
URL:
https://github.com/apache/incubator-heron/issues/2843#issuecomment-378085793
this change went into these three commits along with other changes.
https://github.com/apache/incubator-heron/pull/2840/commits/33dcdcc83bd686a44d01bc2
sreev commented on issue #2843: reduce build time
URL:
https://github.com/apache/incubator-heron/issues/2843#issuecomment-378085793
this change went into these three commits along with other changes.
https://github.com/apache/incubator-heron/pull/2840/commits/33dcdcc83bd686a44d01bc2
kramasamy opened a new pull request #2844: avoid make install for libunwind
URL: https://github.com/apache/incubator-heron/pull/2844
since this requires the dependency on latex2man
This is an automated message from the Apache
jerrypeng closed pull request #2832: allow storm topologies to use the heron
state interface
URL: https://github.com/apache/incubator-heron/pull/2832
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
jerrypeng opened a new pull request #2845: add config for apiserver overrides
URL: https://github.com/apache/incubator-heron/pull/2845
currently a scheduler invoked by the apiserver has no way of getting the set
of config overrides set when launching the apiserver. Adding a config so that
jerrypeng commented on issue #2845: add config for apiserver overrides
URL: https://github.com/apache/incubator-heron/pull/2845#issuecomment-378129092
@cckellogg @kramasamy can you guys take a look? Thanks!
This is an automate
kramasamy closed pull request #2844: avoid make install for libunwind
URL: https://github.com/apache/incubator-heron/pull/2844
This is an automated message from the Apache Git Service.
To respond to the message, please log on
jerrypeng commented on issue #2798: While emitting in spout, adhere to the
batch size limit
URL: https://github.com/apache/incubator-heron/pull/2798#issuecomment-378345732
@nwangtw @maosongfu Is this PR good to go?
This i
kramasamy closed pull request #2798: While emitting in spout, adhere to the
batch size limit
URL: https://github.com/apache/incubator-heron/pull/2798
This is an automated message from the Apache Git Service.
To respond to th
kramasamy closed pull request #2845: add config for apiserver overrides
URL: https://github.com/apache/incubator-heron/pull/2845
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign
ajorgensen commented on issue #2842: Add json format for dry-run
URL: https://github.com/apache/incubator-heron/pull/2842#issuecomment-378367625
I believe the failing test here isn't related to the code I added/modified.
All of the tests run and pass locally.
--
nwangtw opened a new pull request #2846: Nwang/integrate runtime config and
rate limit
URL: https://github.com/apache/incubator-heron/pull/2846
Integrate runtime config with rate limit support.
- Stmgr picks up new configs and update data in physical plan
- Cleanup/refactor/fix ty
nwangtw opened a new pull request #2847: Replace CHECK() with CHECK_*() so that
when a check is failing, more …
URL: https://github.com/apache/incubator-heron/pull/2847
…useful information is logged.
This PR affects only the CHECK() calls in cpp files.
--
nwangtw commented on a change in pull request #2842: Add json format for dry-run
URL: https://github.com/apache/incubator-heron/pull/2842#discussion_r178995626
##
File path:
heron/scheduler-core/src/java/com/twitter/heron/scheduler/dryrun/JsonFormatterUtils.java
##
@@ -0,0
nwangtw commented on a change in pull request #2842: Add json format for dry-run
URL: https://github.com/apache/incubator-heron/pull/2842#discussion_r178994677
##
File path:
heron/scheduler-core/src/java/com/twitter/heron/scheduler/dryrun/JsonFormatterUtils.java
##
@@ -0,0
nwangtw commented on a change in pull request #2842: Add json format for dry-run
URL: https://github.com/apache/incubator-heron/pull/2842#discussion_r178995385
##
File path:
heron/scheduler-core/src/java/com/twitter/heron/scheduler/dryrun/JsonFormatterUtils.java
##
@@ -0,0
nwangtw commented on a change in pull request #2842: Add json format for dry-run
URL: https://github.com/apache/incubator-heron/pull/2842#discussion_r178994343
##
File path:
heron/scheduler-core/src/java/com/twitter/heron/scheduler/dryrun/JsonFormatterUtils.java
##
@@ -0,0
ajorgensen commented on a change in pull request #2842: Add json format for
dry-run
URL: https://github.com/apache/incubator-heron/pull/2842#discussion_r179223844
##
File path:
heron/scheduler-core/src/java/com/twitter/heron/scheduler/dryrun/JsonFormatterUtils.java
##
@@
ajorgensen commented on a change in pull request #2842: Add json format for
dry-run
URL: https://github.com/apache/incubator-heron/pull/2842#discussion_r179224094
##
File path:
heron/scheduler-core/src/java/com/twitter/heron/scheduler/dryrun/JsonFormatterUtils.java
##
@@
ajorgensen commented on a change in pull request #2842: Add json format for
dry-run
URL: https://github.com/apache/incubator-heron/pull/2842#discussion_r179224094
##
File path:
heron/scheduler-core/src/java/com/twitter/heron/scheduler/dryrun/JsonFormatterUtils.java
##
@@
nwangtw commented on issue #2847: Replace CHECK() with CHECK_*() so that when a
check is failing, more …
URL: https://github.com/apache/incubator-heron/pull/2847#issuecomment-378683450
Yeah. It is a good point.
I was changing the safe ones only in this PR to be quick. For the CHECK_E
1 - 100 of 442 matches
Mail list logo