forked from apache/samza
-
Notifications
You must be signed in to change notification settings - Fork 1
Sample Patch #2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
Sanil15
wants to merge
24
commits into
master
Choose a base branch
from
ci-proto
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Sample Patch #2
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This patch adds the capability to register arbitrary timers for both high-level and low-level api. For high-level, InitableFunction will pass the TimerRegistry to user through the new OpContext, and user will implement the TimerFunction to get timer notifications. For low-level api, user can register timer in the TaskContext, and then implements the TimerCallback for specific timer actions. Author: xiliu <xiliu@linkedin.com> Author: xinyuiscool <xinyuliu.us@gmail.com> Author: xinyuiscool <xiliu@linkedin.com> Reviewers: Pateek M <prateekm@gmail.com> Closes apache#419 from xinyuiscool/SAMZA-1498
Tested with ./gradlew clean build, which is passing Author: sanil15 <sanil.jain15@gmail.com> Reviewers: Xinyu Liu <xinyuliu.us@gmail.com> Closes apache#439 from Sanil15/SAMZA-1602
Author: Shanthoosh Venkataraman <svenkataraman@linkedin.com> Reviewers: Xinyu Liu <xinyuliu.us@gmail.com> Closes apache#437 from shanthoosh/fix_zkutils_get_processor_data
…riteCheckpoint Author: Shanthoosh Venkataraman <svenkataraman@linkedin.com> Reviewers: Jagadish <jagadish@apache.org> Closes apache#438 from shanthoosh/SAMZA-1589
Right now when we try to use the SamzaSqlApplicationRunner in yarn mode it goes into the infinite loop because the appRunnerConfig that we try to set is being overwritten by the appRunnerConfig passed by the job and SamzaSqlApplicationRunner keeps creating it again and again in an infinite loop. Fix : Userconfig for app.runner.class should not override the computed one. Added a test case to validate this. Author: Srinivasulu Punuru <spunuru@linkedin.com> Reviewers: Xinyu Liu <xinyuliu.us@gmail.com> Closes apache#440 from srinipunuru/bug-fix.1
Please see commit messages for detailed descriptions. Author: Peng Du <pdu@linkedin.com> Reviewers: Jagadish <jagadish@apache.org>, Wei Song <wsong@linkedin.com> Closes apache#432 from pdu-mn1/remote-table-0222
fix HdfsFileSystemAdapter to get files recursively Author: Hai Lu <halu@linkedin.com> Reviewers: Xinyu Liu <xinyuliu.us@gmail.com> Closes apache#447 from lhaiesp/master
…tor API to compare the offsets Author: Aditya Toomula <atoomula@linkedin.com> Reviewers: Jagadish <jagadish@apache.org> Closes apache#443 from atoomula/bootstrap
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.