Skip to content

Conversation

@tdas
Copy link
Contributor

@tdas tdas commented Apr 24, 2015

This is to ensure that receivers that receive data in small batches (like Kinesis) and want to add them but want the callback function to be called only once. This is for internal use only for improvement to Kinesis Receiver that we are planning to do.

@SparkQA
Copy link

SparkQA commented Apr 25, 2015

Test build #30949 has finished for PR 5695 at commit a7a4cb9.

  • This patch fails Scala style tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch adds the following new dependencies:
    • tachyon-0.6.4.jar
    • tachyon-client-0.6.4.jar
  • This patch removes the following dependencies:
    • tachyon-0.5.0.jar
    • tachyon-client-0.5.0.jar

@huitseeker
Copy link
Contributor

Would it make sense to add the data using the existing ReceivedBlockHandler in this case ? Or is it that you want to reuse the RateLimiter ?

@harishreedharan
Copy link
Contributor

LGTM

@tdas
Copy link
Contributor Author

tdas commented Apr 27, 2015

@huitseeker Yes, I want to use the existing rate limiter interface. So that any sort of rate controlling in the future can be applied through that interface.

@SparkQA
Copy link

SparkQA commented Apr 29, 2015

Test build #31188 has finished for PR 5695 at commit a35cf7d.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@jerryshao
Copy link
Contributor

Looks good to me :).

@tdas
Copy link
Contributor Author

tdas commented Apr 29, 2015

Thanks all! Merging this.

@asfgit asfgit closed this in 5c8f4bd Apr 29, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 14, 2015
… records to BlockGenerator with single callback

This is to ensure that receivers that receive data in small batches (like Kinesis) and want to add them but want the callback function to be called only once. This is for internal use only for improvement to Kinesis Receiver that we are planning to do.

Author: Tathagata Das <tathagata.das1565@gmail.com>

Closes apache#5695 from tdas/SPARK-7138 and squashes the following commits:

a35cf7d [Tathagata Das] Fixed style.
a7a4cb9 [Tathagata Das] Added extra method to BlockGenerator.
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
… records to BlockGenerator with single callback

This is to ensure that receivers that receive data in small batches (like Kinesis) and want to add them but want the callback function to be called only once. This is for internal use only for improvement to Kinesis Receiver that we are planning to do.

Author: Tathagata Das <tathagata.das1565@gmail.com>

Closes apache#5695 from tdas/SPARK-7138 and squashes the following commits:

a35cf7d [Tathagata Das] Fixed style.
a7a4cb9 [Tathagata Das] Added extra method to BlockGenerator.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants