Skip to content
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

add event hub name to the OffsetRange toString function so that it is… #450

Merged
merged 1 commit into from
Jun 27, 2019

Conversation

jgiardin
Copy link
Contributor

… present in the spark metadata. add simple unit test for validation of the string

Thanks for contributing! We appreciate it :)

For a Pull Request to be accepted, you must:

  • Run scalafmt on your code using the .scalafmt.conf present in this project
  • All tests must pass when you run mvn clean test

Just in case, here are some tips that could prove useful when opening a pull request:

  • Read the Contributor's Guide
  • Make the title of pull request is clear and informative.
  • There should be a small number of commits, all with informative messages.
  • The pull request shouldn't introduce any breaking changes (unless will occur on the next release)
  • Any public code should be properly documented
  • Be sure to write tests for any changes in the pull request
  • The code should build without any errors

… present in the spark metadata. add simple unit test for validation of the string
@msftclas
Copy link

msftclas commented Jun 26, 2019

CLA assistant check
All CLA requirements met.

@sjkwak sjkwak merged commit f10a85f into Azure:master Jun 27, 2019
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.

3 participants