svetakvsundhar opened a new issue, #26808:
URL: https://github.com/apache/beam/issues/26808

   ### What needs to happen?
   
   #26604 added an integration test script to a new folder 
`sdks/python/apache_beam/testing/notebooks/`, while current notebooks are all 
under `beam/examples/notebooks/` 
(https://github.com/apache/beam/tree/master/examples/notebooks).
   
   It would make more sense to move the test script here, to allow for the 
`.spec` files, the `.ipynb` notebook, and the test scripts to all be in the 
same folder.
   
   The other (less preferable) alternative, is to add a copy of each notebook 
under `sdks/python/apache_beam/testing/notebooks/`. This would require everyone 
checking in notebooks to check them into `beam/examples/notebooks` as well as 
`sdks/python/apache_beam/testing/notebooks/`, which could eventually lead to 
inconsistency issues between the two.
   
   
   
   ### Issue Priority
   
   Priority: 3 (nice-to-have improvement)
   
   ### Issue Components
   
   - [X] Component: Python SDK
   - [ ] Component: Java SDK
   - [ ] Component: Go SDK
   - [ ] Component: Typescript SDK
   - [ ] Component: IO connector
   - [X] Component: Beam examples
   - [ ] Component: Beam playground
   - [ ] Component: Beam katas
   - [ ] Component: Website
   - [ ] Component: Spark Runner
   - [ ] Component: Flink Runner
   - [ ] Component: Samza Runner
   - [ ] Component: Twister2 Runner
   - [ ] Component: Hazelcast Jet Runner
   - [ ] Component: Google Cloud Dataflow Runner


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to