[GitHub] [samza] cameronlee314 commented on pull request #1493: SAMZA-2647: Clean up unused split deployment code

[GitBox <gi...@apache.org>]

cameronlee314 commented on pull request #1493:
URL: https://github.com/apache/samza/pull/1493#issuecomment-828759493


   > Looks good overall, I see in the testing section that you ran the yarn-integration tests, please ensure that still functional and up to date, I have personally never used it. I just deploy a samza-hello-samza app with a snapshot version.
   > 
   > IIRC was the code not config controlled ? If yes please cleanup any relevant markdowns / docs
   
   I think I did need to update a version locally for the integration tests, but they should otherwise be up to date. I would suggest trying them out, since then you don't need to manually verify anything as long as they report "success".
   The code was config controlled by `job.split.deployment.enabled`, but since the impl wasn't completed, I don't think any docs had been modified to add that yet. I couldn't find any references to the config in the repo.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org