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

MINOR: Fix bug encountered when restoring RUN SCRIPT command. (again) #2265

Merged
merged 2 commits into from
Dec 13, 2018

Conversation

vcrfxia
Copy link
Contributor

@vcrfxia vcrfxia commented Dec 13, 2018

Description

Effectively re-opens #2263, which was reverted as it was merged prematurely.

Testing done

Reviewer checklist

  • Ensure docs are updated if necessary. (eg. if a user visible feature is being added or changed).
  • Ensure relevant issues are linked (description should include text like "Fixes #")

@vcrfxia vcrfxia requested a review from a team as a code owner December 13, 2018 01:10
Copy link
Contributor

@apurvam apurvam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

Copy link
Contributor

@dguy dguy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@apurvam apurvam left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, again!

@vcrfxia vcrfxia merged commit 8e06d7a into confluentinc:5.1.x Dec 13, 2018
hasnat added a commit to hasnat/ksql that referenced this pull request Dec 23, 2018
* fork/master: (107 commits)
  copy-edited (confluentinc#2299)
  copy-edited (confluentinc#2299)
  remove unused map from command store `getRestoreCommands` (confluentinc#2296)
  Speed up integration tests. (confluentinc#2288)
  Move KSQL Clickstream demo to Examples repo (confluentinc#2270)
  Bump Confluent to 5.1.1-SNAPSHOT, Kafka to 2.1.1-SNAPSHOT
  Set Confluent to 5.1.0, Kafka to 2.1.0-cp1.
  Add ServiceContext (confluentinc#2243)
  Update to use CCL (confluentinc#2278)
  Remove redundant intro sentence in README.md (confluentinc#2277)
  Switch to use CCL (confluentinc#2275)
  Update readme for CCL (confluentinc#2276)
  Minor: add Hamcrest matchers for KsqlRestException and KsqlErrorMessage (confluentinc#2273)
  Update per relicensing (confluentinc#2274)
  CP-584: 5.1.0 changelog (confluentinc#2267)
  add null checks to min, max, and count aggregates; call Math.min/max for consistency (confluentinc#2246)
  Return and accept command topic offsets via REST API (confluentinc#2159)
  MINOR: Fix bug encountered when restoring RUN SCRIPT command. (again) (confluentinc#2265)
  More improvements to the way the CLI handles inline comments in multi-line statements. (confluentinc#2241)
  Fix issue where Avro subject does not exist (confluentinc#2260)
  ...
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