ByteStreamCnvSvc: Fix flake8 py3 warnings.

• file -> open; use binary mode for pickles.
• trailing semicolons.
• unneeded imports
• not ... in ... -> ... not in ...
• exception raising syntax
• has_key -> in

added 197 commits

• 740174ee...47ecfc89 - 196 commits from branch atlas:master
• 6c0f5b27 - Merge branch 'master' into 'py3.ByteStreamCnvSvc-20191202'

Compare with previous version

This merge request affects 1 package:

• Event/ByteStreamCnvSvc

added EDM master review-pending-level-1 labels

CI Result FAILURE (hash 6c0f5b27)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 6383]

Test timed out.

Jenkins please retry the build

Jenkins please retry a build

This merge request affects 1 package:

• Event/ByteStreamCnvSvc

CI Result SUCCESS (hash 6c0f5b27)

	Athena	AthSimulation
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 6438]

No outstanding issue, moving forward with approval. -L1

added review-approved label and removed review-pending-level-1 label

merged

mentioned in commit 97f5a3c6

added sweep:ignore label